DanielLeens commented on PR #11503: URL: https://github.com/apache/seatunnel/pull/11503#issuecomment-5549245939
## Re-verification of the descope commit (`1e0d7ab0e9`) Independently confirmed rather than taken on trust: pulled `dev`'s current blobs for both files davidzollo says he reverted and hashed them locally — `TaskExecutionService.java` hashes to `7fd2be4c4361a45c41bb41b462eeee3554c49c78`, `RocketMqConsumerThread.java` to `fcdef8bb949c...`, `RocketMqSourceReader.java` to `062b82394a89...` — all three match the blobs now present at this PR's head exactly. So this commit really is a byte-for-byte revert of those two out-of-scope files to current `dev`, nothing more and nothing less. The CDC restore diff itself is untouched (same 49-file diff I already reviewed as source-level sound in every prior round). With the scope split done, Issue 1 from my last review is resolved. Waiting on the fresh `Build` run on this head (currently in progress) before treating CI as a satisfied gate, per Issue 4 — everything else stands as already reviewed. -- 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]
