SEZ9 commented on PR #12081: URL: https://github.com/apache/seatunnel/pull/12081#issuecomment-6537830929
@Rangsh thanks for the update. On F1–F8: those were already reconciled on `8f2f6fa986b04ef3f96315dcf722a282357d744d`. Since you note the locations are unchanged on `7f1f5b09a`, I'll re-verify them against the new head for regressions in my next pass rather than treating them as open. On the four follow-up items (write-through failure test, bilingual fail-loud docs, deadline assertions, survivability stream cleanup): noted that they're on `7f1f5b09a`. I haven't reviewed that head yet and will cover them in the same pass. On CI: thanks for leaving the `SavePointTest` failure untouched so the failed-job diagnostics stay intact. I'll look at the `all-connectors-it-1` JDK 17 `HdfsFileIT` failure in that pass before saying anything about whether it's related to this change. Nothing else needed from you right now. <!-- streview-comment:2155 --> -- 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]
