DanielLeens commented on PR #13126: URL: https://github.com/apache/seatunnel/pull/13126#issuecomment-6699531417
Thanks @SEPURI-SAI-KRISHNA, that closes the last open point from my review. - #13155 now tracks the Spark local-mode hang, and the exclusion in `CassandraIT.java:147-160` points to a concrete follow-up, so it is traceable. - I checked [fork run 42075267404](https://github.com/SEPURI-SAI-KRISHNA/seatunnel/actions/runs/42075267404): it ran on `320ac7c6d6` and concluded `success`. The apache `Build` check on this head is green as well. - Moving the Spark fix away from closing in `abort()` and into a separate PR (#13156) makes sense. I'll review it there on its own merits and won't fold it into this PR. Nothing else is needed on this PR from my side. My source-review conclusion on this head is the same: no blockers. A maintainer with write access still has to give the approval and do the merge. -- 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]
