-
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
[debezium] Upgrade to Debezium 1.9.8.Final #3034
Conversation
@leonardBang , @gunnarmorling , CC |
@@ -35,7 +35,7 @@ | |||
import java.util.regex.Pattern; | |||
|
|||
/** | |||
* 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 |
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.