-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[FLINK-35670][cdc-connector][postgres]Flink cdc pipeline support postgres source #3968
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Hi @Gunnnn. We plan to complete it in next version(3.5.0). |
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...rc/test/java/org/apache/flink/cdc/connectors/postgres/source/PostgresPipelineITCaseTest.java
Outdated
Show resolved
Hide resolved
Hi, @Mrart. |
And a document for this is necessary. |
Thanks for review, let me optimize it |
@Mrart There is an error in checking style.
|
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Outdated
Show resolved
Hide resolved
...-pipeline-e2e-tests/src/test/java/org/apache/flink/cdc/pipeline/tests/PostgresE2eITCase.java
Outdated
Show resolved
Hide resolved
Seems newly added test cases are failing, could @Mrart take a look? |
It seems the test env problems, we can trigger test again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have left some message.
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Show resolved
Hide resolved
...src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresDataSourceOptions.java
Show resolved
Hide resolved
...postgres/src/main/java/org/apache/flink/cdc/connectors/postgres/utils/PostgresTypeUtils.java
Show resolved
Hide resolved
@lvyanquan After CI passes, is it possible to enter the next process? |
docs/content.zh/docs/connectors/pipeline-connectors/postgres.md
Outdated
Show resolved
Hide resolved
docs/content.zh/docs/connectors/pipeline-connectors/postgres.md
Outdated
Show resolved
Hide resolved
.../src/main/java/org/apache/flink/cdc/connectors/postgres/source/PostgresMetadataAccessor.java
Outdated
Show resolved
Hide resolved
...va/org/apache/flink/cdc/connectors/postgres/source/reader/PostgresPipelineRecordEmitter.java
Outdated
Show resolved
Hide resolved
...or-postgres-cdc/src/main/java/org/apache/flink/cdc/connectors/postgres/PostgreSQLSource.java
Outdated
Show resolved
Hide resolved
...java/org/apache/flink/cdc/connectors/postgres/source/config/PostgresSourceConfigFactory.java
Outdated
Show resolved
Hide resolved
Thanks @Mrart for this contribution, merged. |
Thanks @Mrart and all who involved this feature for the great work ! |
…pache#3968) PostgreSQL Pipeline Source does not support synchronizing table structure changes currently. --------- Co-authored-by: ChengJie1053 <[email protected]>
Fork from : #3442
https://issues.apache.org/jira/browse/FLINK-35670