Skip to content

Conversation

@loserwang1024
Copy link
Contributor

@loserwang1024 loserwang1024 commented Apr 3, 2025

Currenly, tests are often stuck in removing table test . The reason is:

  1. when getting all the data of snapshot split, the assigned status maybe still not switched from INITIAL_ASSIGNING to INITIAL_ASSIGNING_FINISHED. (If stop and restart job now, the table won't be removed from enumerator.)
  2. when restarted, the job is still be snapshot phase, and table will be read again. It is not what the testRemoveTablesOneByOne wants.
  3. Disable enable backfill test in sqlserver cdc connector until FLINK-34833 is resolved

Thus, this pr will wait 1s until the assigned status is INITIAL_ASSIGNING_FINISHED.

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.

+1

@loserwang1024
Copy link
Contributor Author

The mysql test is failed and I fixed it, run it sucessfule in my CI, https://github.com/loserwang1024/flink-cdc-connectors/actions/runs/14237185479
image

@leonardBang , Could you please help to rerun this mysql test after my modification.

@leonardBang
Copy link
Contributor

The mysql test is failed and I fixed it, run it sucessfule in my CI, https://github.com/loserwang1024/flink-cdc-connectors/actions/runs/14237185479 image

@leonardBang , Could you please help to rerun this mysql test after my modification.

Sure

…INITIAL_ASSIGNING_FINISHED so that newly added table won't be stuck.
@leonardBang leonardBang merged commit ec1e0ab into apache:master Apr 10, 2025
26 checks passed
SML0127 pushed a commit to SML0127/flink-cdc-connectors that referenced this pull request May 26, 2025
…NG_FINISHED so that newly added table won't be stuck

This closes apache#3978
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants