Skip to content
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

[flink] support sync multiple mysql database to paimon #1282

Closed
wants to merge 1 commit into from

Conversation

zhangjun0x01
Copy link
Contributor

@zhangjun0x01 zhangjun0x01 commented Jun 2, 2023

Purpose

Linked issue: close #1238

Users can synchronize one or multiple MySQL database into one or multiple Paimon database, It is default false, in this case, all the MySQL database will be synchronized to one paimon database, and the table with same name in different database will be merged, it is suitable for database sharding scenarios. if it is true, the parameter "--database" will be ignored, all the MySQL database will be synchronize to multiple paimon database with the same schema as MySQL before, it is suitable for scenarios with a large number of databases under a database instance, which can save resources.

for new added table in the sync database runtime , I need to get the databaseName and tableName ,otherwise the test case will be failed, but the druid framework can not get the databaseName, so I merge PR #1621 to this PR, so we should review PR #1621 first , when it is merged to master , I will rebase from the master .

Tests

org.apache.paimon.flink.action.cdc.mysql.MySqlSyncDatabaseActionITCase

API and Format

Documentation

@zhangjun0x01 zhangjun0x01 marked this pull request as draft July 19, 2023 15:32
@zhangjun0x01 zhangjun0x01 force-pushed the multiple_db_cdc branch 2 times, most recently from 72266b1 to 65363c5 Compare July 21, 2023 06:38
@zhangjun0x01 zhangjun0x01 marked this pull request as ready for review July 21, 2023 06:39
@zhangjun0x01 zhangjun0x01 marked this pull request as draft July 21, 2023 06:40
@zhangjun0x01 zhangjun0x01 force-pushed the multiple_db_cdc branch 2 times, most recently from 1791d3c to 1033342 Compare July 21, 2023 09:48
@zhangjun0x01 zhangjun0x01 marked this pull request as ready for review July 21, 2023 09:49
@zhangjun0x01 zhangjun0x01 force-pushed the multiple_db_cdc branch 3 times, most recently from 50df986 to b34546d Compare July 23, 2023 17:01
@zhangjun0x01 zhangjun0x01 force-pushed the multiple_db_cdc branch 2 times, most recently from e071aaf to ee03fc5 Compare July 30, 2023 05:14
@JingsongLi
Copy link
Contributor

Can you rebase latest master?

@zhangjun0x01 zhangjun0x01 marked this pull request as draft August 2, 2023 15:47
@zhuangchong
Copy link
Contributor

Already supported, I will close this PR first. If you have any questions, you can open it again.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] support multiple database sync for flink cdc sync action
3 participants