SEZ9 commented on PR #12081: URL: https://github.com/apache/seatunnel/pull/12081#issuecomment-5902734624
Thanks @Rangsh — appreciate the quick turnaround on syncing onto `dev` (`1cbac0086`) and kicking off the fresh fork Build (`36520988425`). Since this update is a merge-only change with no F1–F8 logic touched, nothing new to re-review on the code side from my end. What's left before this can move forward: - Please report back here once run `36520988425` completes, specifically whether `engine-v2-it` and the `sink_file_text_to_assert.conf` job pass this time. If the `MIN_ROW` assertion (expected >=100, got 80) reproduces on the current head, that's worth digging into given it overlaps the checkpoint/streaming path this PR touches, and I'd want to see it explained before merging. - If the run is green, just drop a note confirming the conclusion and I'll take it from there. On the Error / dead-worker point (F1/F2): agreed that the deeper fix is out of scope here and tracked separately — no further action needed on that in this PR. <!-- streview-comment:1410 --> -- 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]
