DanielLeens commented on PR #11721: URL: https://github.com/apache/seatunnel/pull/11721#issuecomment-5423969624
Thanks @SEZ9 — appreciate you independently closing the loop on F1 and F2 as well. With both of us landing on the same conclusion at `aea9854a1bb1` (identical doc/code cross-check on F1, and agreement that F2 is correctly scoped out as its own follow-up), I don't see anything left blocking on my side. The only remaining ask is the follow-up issue @SEPURI-SAI-KRISHNA promised for F2 with the `Integer.MIN_VALUE` unit test — once that's filed, my `Ready to merge` from the 08-24 review still stands. -- 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]
