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

[Bug][cdc-connectors][cdc-base] Transaction log of high_watermark read twice #2885

Closed
2 tasks done
loserwang1024 opened this issue Dec 18, 2023 · 4 comments
Closed
2 tasks done
Labels
bug Something isn't working

Comments

@loserwang1024
Copy link
Contributor

loserwang1024 commented Dec 18, 2023

Search before asking

  • I searched in the issues and found nothing similar.

Flink version

1.18

Flink CDC version

3.0

Database and its version

anyway

Minimal reproduce step

Reason

Current, read snapshot split then backfill between [low_watermark, high_watermark], then read in stream phase between [high_watermark, +∞). The message of high_watermark will be read twice.

What did you see instead?

Anything else?

No response

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@link3280
Copy link
Contributor

@loserwang1024 Hi, does this issue affect mysql-cdc as well (since it involves cdc-base)? Thanks!

@loserwang1024
Copy link
Contributor Author

@link3280,just a minor optimization. To be honest, this rarely happens because high_watermark is mostly non dml record, such as heartbeat record.

@link3280
Copy link
Contributor

@loserwang1024 Thanks a lot for your input! The reason why I ask is that I met a data duplication issue with MySQL CDC 3.0.0 when it reads binlogs with the position set to earliest-offset or timestamp. The data showed up twice exactly. I checked the logs and all the splits were MySqlBinlogSplit, so the logs may be read twice. I wonder if it's the same issue.

@PatrickRen
Copy link
Contributor

Closing this issue as it has been migrated to Apache Jira.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants