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

[hotfix] remove unused code in PR#2220 #2231

Merged
merged 3 commits into from
Jun 25, 2023

Conversation

wallkop
Copy link
Contributor

@wallkop wallkop commented Jun 21, 2023

I must say I'm very sorry that I inadvertently left several lines of unused code in previous PR: #2220. This code was just for debugging, but ended up being inadvertently forgotten to clean up.

In order to ensure better readability of the 2.4.0 code, I am here to initiate a new PR to clean up the relevant code.

Copy link
Contributor

@ruanhang1993 ruanhang1993 left a comment

Choose a reason for hiding this comment

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

LGTM.

@ruanhang1993 ruanhang1993 merged commit e088d01 into apache:master Jun 25, 2023
15 checks passed
yuxiqian added a commit to yuxiqian/flink-cdc that referenced this pull request Jun 30, 2023
…-2062

* origin/feat/issue-2062:
  [docs] Update connector link to 2.5-SNAPSHOT in docs
  [build] Bump version to 2.5-SNAPSHOT
  [hotfix][mysql] remove unused code (apache#2231)
  [hotfix] Add vitess connector to the release profile (apache#2233)
  [docs][hotfix] Update debezium reference links to 1.9 version
  [build] Update the copyright year to 2023 (apache#2205)
  [postgres] Fix postgres e2e test
  [postgres] scan.incremental.snapshot.enabled is closed by default
  [postgres] Backfill task will be able to end when there is not new change data but read the ending lsn
  [postgres] Create slot for backfill task before snapshot reading
  [postgres] Prepare a slot for the unique global stream split
  [mysql] Fix GTID issues to recover from checkpoint normally in specifying startup mode (apache#2220)
ruanhang1993 pushed a commit to ruanhang1993/flink-cdc-connectors that referenced this pull request Jul 20, 2023
zhongqishang pushed a commit to zhongqishang/flink-cdc that referenced this pull request Dec 7, 2023
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