goutamadwant commented on PR #11512: URL: https://github.com/apache/seatunnel/pull/11512#issuecomment-5234130787
@SEZ9 thanks again for the review! I rechecked the current head against these points. The public CDC progress types already have @Experimental and both English and Chinese docs clarify that this data stays inside the engine for now and is not exposed through REST CLI or metrics The implementation also uses the common CdcProgressProvider and CdcProgressReport contracts with one tagged CdcProgressEnvelope one operation payload and one role keyed service map for reader and enumerator reports The latest CI failures are WorkerTagTest failing to start a Hazelcast node on Windows and PaimonSinkCDCIT timing out on an existing sink writer thread. The CDC progress tests passed in the same run and neither failing path is changed by this PR Could you please recheck the latest head and let me know if any specific change is still missing cc @DanielLeens -- 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]
