SEZ9 commented on PR #12454: URL: https://github.com/apache/seatunnel/pull/12454#issuecomment-5841962262
Thanks @goutamadwant for the stress-test comparison. I checked the current patches: #12449 confines the checkpoint injection fix to the existing E2E test, and #12444 fixes the response-side latch ordering in the existing PayPal test helper. Both are still open, as is #12216. I agree that the smaller fixes should take precedence when they land. For now I am retaining the validated version of this PR; once those fixes are in `dev`, I will rebase and remove the superseded checkpoint accessor/test and PayPal hunks, then rerun validation. The existing green run only covers the current head, not that future reduced diff. The shared patches have already been removed from #12298 and #12299. Their remaining OceanBase `TableId` serialization and MinIO image-access failures are separate from these overlapping fixes, so merging this PR alone is not a guarantee that both feature PRs become green. @DanielLeens <!-- seatunnel-common-overlap-followup-20260926 --> -- 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]
