Vivek1106-04 commented on issue #12339: URL: https://github.com/apache/seatunnel/issues/12339#issuecomment-5756763123
Ran `MysqlCDCWithSchemaChangeIT` against the branch, as promised above. ``` Tests run: 4, Failures: 0, Errors: 0, Skipped: 0 (276s) ``` All four Zeta cases: schema evolution, exactly-once schema evolution, multi-database same-table evolution, and the schema-change event type filter. Across 96 schema-change checkpoint triggers there were no `CHECKPOINT_EXPIRED` events, no timed-out checkpoints, no skipped generic checkpoints, and no coordinator errors. So the extra queueing hop on a zero-delay schema-change trigger does not show up on this path. Caveat worth stating: this is one local run on an unloaded host. It shows no regression, but it does not exercise a saturated dispatch pool, which is exactly what the scheduling-delay metric is for. Separately, the `thenAcceptAsync` fix from point 1 above is now pushed to #12165 as `31a454f35`, with a test that completes the pending checkpoint future before registration and asserts the body does not run on the registering thread. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
