-
Notifications
You must be signed in to change notification settings - Fork 523
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
fix: mysql cdc read timezone and correct mysql test image #16732
Conversation
java/connector-node/risingwave-connector-service/src/main/resources/mysql.properties
Show resolved
Hide resolved
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.
system ok | ||
mysql -e " | ||
SET GLOBAL time_zone = '+01:00'; | ||
" | ||
|
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 will actually affect other potential mysql test running in parallel, do you have any suggestion on addressing this? @xxchan
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.
If really needed, you can write the old value to some tmp file, and restore it at the end.
I think many source tests can only be run in serial...
I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
The bug will affect Mysql CDC if the MySQL table time zone is not UTC
The default param
connectionTimeZone
will be server. That mean it will convert time column by the session timezone param. If set to '+00:00', debezium will assume the server timezone is '+00:00'Since there will be corner cases for
connectionTimeZone=SERVER
, we will useconnectionTimeZone=+00:00
andforceConnectionTimeZoneToSession=true
close #16496
Checklist
./risedev check
(or alias,./risedev c
)Documentation
Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.