-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Cherry pick bugfix commits for Release 2.4.2 #2591
Conversation
![]() |
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.
Thanks @GOODBOY008 for the great work, I left some comments about API change commits, I think we should only introduce the bugfix commits for bugfix version.
* produces full changelog mode structure {@link RowData}. | ||
*/ | ||
@PublicEvolving | ||
public class MongoDBConnectorFullChangelogDeserializationSchema |
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.
This is a feature with public API change, I'm not sure we can pick this or not, I tend to skip this one
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.
Thanks @leonardBang, it makes sense to me.
The supports of MongoDB 6.0 will indeed affect the API, but it will maintain compatibility with the old version under the default configuration. I tend to skip this one as well.
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.
cc @GOODBOY008.
* <p>See more | ||
* https://cwiki.apache.org/confluence/display/FLINK/FLIP-147%3A+Support+Checkpoints+After+Tasks+Finished. | ||
*/ | ||
public PostgresSourceBuilder<T> closeIdleReaders(boolean closeIdleReaders) { |
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.
This should be a API change feature?
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.
There is no API change, here is something omitted here at 2.3
/** VTCtld server config. */ | ||
public Builder<T> vtctldConfig(VtctldConfig vtctldConfig) { | ||
this.vtctldConfig = vtctldConfig; | ||
/** |
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.
Looks like we pick some features
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.
These changes belong to [vitess] Migrate connector properties to debezium 1.9.7 (#2260)
. It's ok to merge this commit.
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 understand your concern and drop this commit.
@leonardBang About mongodb hotfix, I added under three commits because upper bugfix depend on them. I confirmed with @Jiabao-Sun , he think it's ok to add this three commits. |
@GOODBOY008 @Jiabao-Sun Keep API compatible is pretty important, we need ensure that all users who use Flink CDC 2.4.0& 2.4.1 can bump to Flink CDC 2.4.2 without any SQL/code change, I'm not sure this PR would break this constraint or not, looks like this PR would break this? |
d239f72
to
473e749
Compare
…xx = database, schema or table) (apache#2212) (cherry picked from commit ba5fcbc)
(cherry picked from commit 0714688)
This closes apache#2334. (cherry picked from commit 883b684)
…m for Oracle CDC Connector This closes apache#2325. Co-authored-by: skylines <34996528+gtk96@users.noreply.github.com> (cherry picked from commit 65bdbfd)
(cherry picked from commit 6c2fe08)
(cherry picked from commit bca60a4)
(cherry picked from commit 0b9ca59)
This closes apache#2345. (cherry picked from commit 61b9a2a)
…ailedSource (apache#2384) * [hotfix][mysql] fix failed test LegacyMySqlSourceTest#testSnapshotOnFailedSource * [hotfix][mysql] fix failed test LegacyMySqlSourceTest#testSnapshotOnFailedSource (cherry picked from commit b8f468e)
This closes #apache#2362. Co-authored-by: TJX2014 <tangjinxin@deepexi.com> (cherry picked from commit 094fff4)
…pleTest#testConsumingAllEvents (apache#2364) (cherry picked from commit a3ddc48)
apache#2388) Co-authored-by: DESKTOP-TQDONF3\狗狗jian <zhangjian@adream.org> (cherry picked from commit f7df47e)
…ner reuse bug (apache#2394) (cherry picked from commit 90ed963)
…tikv disk full` failure (apache#2402) (cherry picked from commit 8098c82)
… tables (apache#2160) (cherry picked from commit 92aca18)
(cherry picked from commit 420fe11)
… parsing MySQL DDL (apache#2418) (cherry picked from commit 8e7378d)
* [sqlserver] Fix backfill stream task hang * [sqlserver] Fix backfill stream task hang (cherry picked from commit 6fa02ef)
* [oracle] Correct the naming error * Fix comments * fix name error (cherry picked from commit 8feb51b)
…able name with dot (apache#2443) (cherry picked from commit 7801ac9)
…pache#2399) (cherry picked from commit 84e97df)
… calculation This closes apache#2421. Co-authored-by: Leonard Xu <leonard@apache.org> (cherry picked from commit dddab4d)
(cherry picked from commit 07db169)
(cherry picked from commit 7172695)
…stream split (apache#2425) (cherry picked from commit 82e3053)
(cherry picked from commit 928ccf1)
473e749
to
31b7b79
Compare
![]() |
) (cherry picked from commit aec2a91)
e850811
to
92e642f
Compare
LGTM, +1 |
The CI failed even I triggered retry, @GOODBOY008 @lvyanquan could you take a look ? |
Yes,I am trying to find out reason. |
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.
Thank all for this PR, especially thanks @GOODBOY008 for preparing the PR and
troubleshooting the CI failure, LGTM
Cherry pick bugfix commits for Release 2.4.2.
![69dcbabed4ace81837e139ca50a8d718](https://private-user-images.githubusercontent.com/13617900/278932786-d33de617-3ce7-43b7-9798-af0af521c362.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkwNzYxNDYsIm5iZiI6MTczOTA3NTg0NiwicGF0aCI6Ii8xMzYxNzkwMC8yNzg5MzI3ODYtZDMzZGU2MTctM2NlNy00M2I3LTk3OTgtYWYwYWY1MjFjMzYyLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjA5VDA0MzcyNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTQwYWZhZmYwZWU0ZmU5OTliODU3MDJiYjEzMTczYmI0N2U2MzI2ZDA1NGY2MzBiMzFiN2UyYjViNjZmOGNmZjEmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.ahQEcPIw_3CKhGF-mMzPm65mUyYjOZJXY-jDzB87CH8)
![58653833d8870608be3bdfc4084912d6](https://private-user-images.githubusercontent.com/13617900/278932811-376b25dd-0ad9-47ed-a860-a470327f2a39.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkwNzYxNDYsIm5iZiI6MTczOTA3NTg0NiwicGF0aCI6Ii8xMzYxNzkwMC8yNzg5MzI4MTEtMzc2YjI1ZGQtMGFkOS00N2VkLWE4NjAtYTQ3MDMyN2YyYTM5LnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjA5VDA0MzcyNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWNiNDQzYjIyZjVjNTQyNmNjZGQxYjQ3ZTcwZDExYmQzMWU1ZDE5NjZjNDkyYzAwMDY1ZWY2MGJlNDQwMjg5NGQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.fhzdbs32pLKN6UX5Yy6YPDQ2g6MlTCveLx-FTsSNrbM)
![497defa6704a6ce8d20d355d379f236b](https://private-user-images.githubusercontent.com/13617900/278932872-0228f3f4-4c7a-4056-89c8-747ea6e496b3.png?jwt=eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3MzkwNzYxNDYsIm5iZiI6MTczOTA3NTg0NiwicGF0aCI6Ii8xMzYxNzkwMC8yNzg5MzI4NzItMDIyOGYzZjQtNGM3YS00MDU2LTg5YzgtNzQ3ZWE2ZTQ5NmIzLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNTAyMDklMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjUwMjA5VDA0MzcyNlomWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPWIyNWEwMTIzMmY4ZDMxMWM5YTBlM2E2M2ZiNDBkNWRmZTUyNDE1ZDQ1NWFjMzFhZmNmMjU2NTYwYjAzMzgxMGQmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0In0.JFchinupba7Vy4yzmUiwDUXG5F-KV7_1GJNbvmqZI1Y)