SEPURI-SAI-KRISHNA commented on PR #12290: URL: https://github.com/apache/seatunnel/pull/12290#issuecomment-5758392838
Following up on the #12377 note above: I read that thread properly and @SEZ9 already raised exactly that point in a changes-requested review on 09-18, including the `SubPlan.java:219` transition out of CANCELING and the regression test bypassing it. There is nothing for me to add there. That also sharpens the answer to your question. Both engine failures already have fixes written. #12311 covers the cancellation one, and the #12353 thread has settled on it as the implementation rather than a separate PR. #12313 or #12316 covers the backpressure one. None of them is waiting on analysis. #12311 and #12313 have no approvals, and #12377 has had blocking review comments unanswered since 09-18. Review attention is what would move them, so that is where I would put any help you want from me. -- 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]
