DanielLeens commented on PR #11909: URL: https://github.com/apache/seatunnel/pull/11909#issuecomment-5437626327
Thanks for confirming, @davidzollo. Agreed from my side as well — this one is ready to merge. The two Medium issues from my earlier review (the forwarded-path master-transition exception classification, and the `dynamicSlot` `/pending-jobs` `totalSlots`/`freeSlots` accounting) were fixed and verified against `9247b2dc3` with the added regression tests, so there are no remaining blockers on the code side. -- 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]
