Skip to content
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

There is a typo in the comments in MysqlSnapshotSplit. #1513

Merged
merged 1 commit into from
Aug 29, 2022
Merged

There is a typo in the comments in MysqlSnapshotSplit. #1513

merged 1 commit into from
Aug 29, 2022

Conversation

empcl
Copy link
Contributor

@empcl empcl commented Aug 28, 2022

should be "The high watermark is not null when the split read finished.", but not 'bull'.

Copy link
Contributor

@leonardBang leonardBang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@leonardBang leonardBang merged commit cf8a2f4 into apache:master Aug 29, 2022
fuyun2024 pushed a commit to fuyun2024/flink-cdc-connectors that referenced this pull request Sep 22, 2022
Co-authored-by: chenlei677 <chenlei677@jd.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants