Skip to content

Conversation

yuanoOo
Copy link
Contributor

@yuanoOo yuanoOo commented Jul 21, 2025

Background

  • Log Proxy is officially deprecated, making the existing OceanBase CDC connector obsolete.
  • OceanBase now provides OceanBase Binlog Service, a more stable and scalable solution for CDC.

Proposed Changes

  • Remove Legacy Code:
    • Delete all code related to the deprecated LogProxy-based implementation.
  • Base on mysql-cdc:
    • Refactor the OceanBase CDC connector by leveraging the existing mysql-cdc connector.
    • Adapt the codebase to ensure compatibility with OceanBase Binlog Service.
  • Documentation Update:
    • Highlight differences between the old (LogProxy) and new (Binlog Service) implementations.

@github-actions github-actions bot added docs Improvements or additions to documentation e2e-tests oceanbase-cdc-connector labels Jul 21, 2025
@yuanoOo
Copy link
Contributor Author

yuanoOo commented Jul 21, 2025

@lvyanquan Please help trigger github CI and take a look.

@yuanoOo
Copy link
Contributor Author

yuanoOo commented Jul 21, 2025

@sunzhaoyang Please review the doc.

@yuanoOo
Copy link
Contributor Author

yuanoOo commented Aug 12, 2025

@lvyanquan Hi, I checked the reason for the GitHub CI test failure, and it appears to be unrelated to this PR.

@lvyanquan
Copy link
Contributor

@lvyanquan Hi, I checked the reason for the GitHub CI test failure, and it appears to be unrelated to this PR.

Yes, you can rebase the master to solve this literary problem.

@yuanoOo yuanoOo force-pushed the refactor-oceanbase-cdc branch from 5a0fec9 to 96f4934 Compare August 12, 2025 07:06
@yuanoOo yuanoOo force-pushed the refactor-oceanbase-cdc branch from 96f4934 to b97bd33 Compare August 12, 2025 07:16
@yuanoOo
Copy link
Contributor Author

yuanoOo commented Aug 12, 2025

@lvyanquan Hi, I checked the reason for the GitHub CI test failure, and it appears to be unrelated to this PR.

Yes, you can rebase the master to solve this literary problem.

@lvyanquan I've completed the rebase. Please help trigger GitHub CI again and take a look.

@lvyanquan lvyanquan added the 3.5 label Aug 14, 2025
Copy link
Contributor

@lvyanquan lvyanquan left a comment

Choose a reason for hiding this comment

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

LGTM.

@lvyanquan lvyanquan merged commit fea9807 into apache:master Aug 15, 2025
63 of 64 checks passed
Mrart pushed a commit to Mrart/flink-cdc that referenced this pull request Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants