-
Notifications
You must be signed in to change notification settings - Fork 2.1k
[debezium] Upgrade to Debezium 1.9.8.Final #3034
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
|
@leonardBang , @gunnarmorling , CC |
| /** | ||
| * Copied from Debezium project(1.9.7.final) to add BIGINT and SMALLINT to TRIM_DATA_TYPES. Remove | ||
| * Copied from Debezium project(1.9.8.final) to add BIGINT and SMALLINT to TRIM_DATA_TYPES. Remove | ||
| * this when https://issues.redhat.com/browse/DBZ-6824 is fixed. |
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.
FYI As far I can see from the issue's comments, this issue is closed since Debezium 2.3
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 a lot, it seems it is fixed in 2.3.3.Final. Add this to the JavaDocs,
7154ff9 to
e05fa8f
Compare
e05fa8f to
9e05132
Compare
|
@GOODBOY008 , CC |
GOODBOY008
left a comment
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.
LGTM @loserwang1024 Thank you for driving this issue.
As shown in #2899, the Debezium team released another 1.9.x release the other day with a few bug fixes, including support for ingesting changes from Postgres 16 read replicas. Thus, update to 1.9.8.Final therefore.