DanielLeens commented on PR #11721: URL: https://github.com/apache/seatunnel/pull/11721#issuecomment-5583655120
@SEPURI-SAI-KRISHNA thanks for the detailed pointers on both — I independently re-verified against the current head () in a scratch checkout rather than taking the claims at face value. **F1 confirmed.** In , the reproduction now reads , , and , followed by — matching the real on this head token-for-token. The one-line summary snippet carries the same masked expression, and the old callout is now a explaining the mask-vs- distinction. does appear on the page, but only in the field listing and the capacity-planning prose — never inside a routing code block — and appears nowhere on the page or in . So: the snippet genuinely mirrors the implementation now, not just a callout removal. **F2 confirmed.** doesn't exist anywhere in this PR's tree at . It landed on via #11987 on 2026-09-01 — a week after this branch's last merge on 08-24 — and its is exactly , the same formula this PR keeps inline at . #11976 is confirmed still open and tracks the migration. Worth flagging for the thread record: F1 and F2 are the same two items the three of us already closed out on 08-26/08-27 at this exact head — the code hasn't moved since then, so this is a re-confirmation of an already-resolved state rather than new ground. Nothing changes on my side: my 08-24 approval still stands against the current code, no blockers remain from my pass. -- 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]
