-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[mysql] Use local timezone as the default server timezone #1407
Conversation
Spotless failed. It seems you should run |
...ysql-cdc/src/main/java/com/ververica/cdc/connectors/mysql/table/MySqlTableSourceFactory.java
Outdated
Show resolved
Hide resolved
...l-cdc/src/main/java/com/ververica/cdc/connectors/mysql/source/config/MySqlSourceOptions.java
Outdated
Show resolved
Hide resolved
OracleE2eITCase.testOracleCDC failed, but it seems unrelated to the changes made. |
@kylemeow Thanks for the contribution, I've fixed the test, could you rebase master and try again? |
76f9fc5
to
8850631
Compare
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 @kylemeow for the contribution, I left some comments
...l-cdc/src/main/java/com/ververica/cdc/connectors/mysql/source/config/MySqlSourceOptions.java
Show resolved
Hide resolved
b9476d5
to
828c018
Compare
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 @kylemeow for the contribution, I left a few comments
flink-connector-mysql-cdc/src/test/java/com/ververica/cdc/connectors/mysql/MySqlTestUtils.java
Outdated
Show resolved
Hide resolved
flink-connector-mysql-cdc/src/main/java/com/ververica/cdc/connectors/mysql/MySqlValidator.java
Outdated
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 @kylemeow for the nice work, so happy to discuss with you, LGTM.
… option (apache#1407) This closes apache#1406.
Due to the issue mentioned in #1406, Flink CDC Connectors might treat Timestamp fields with different behaviors during snapshot and streaming phases.
Here I propose changing the default value of
server-time-zone
to local timezone so that most users would not need to manually set timezone to get the correct results.