DanielLeens commented on PR #12173: URL: https://github.com/apache/seatunnel/pull/12173#issuecomment-5617087919
Thanks @Rangsh, that per-item mapping matches what I verified independently against the current head (`c0d59da54`): items 2, 4 and 5 (locking, Javadoc, tests) landed in `ca82f95d3b`, item 3 (docs) in `909f878402`, and item 1 (durability sampling) is the first-iteration + trial-teardown restructuring from `ca82f95d3b`/`bf1110b160`/`3794d03e44`, with `c0d59da54` only tightening the Javadoc wording on how much of the batch that sample actually proves. CI on this exact head is green (fork `Build` run `34355079803`, conclusion success), and `mergeStateStatus` is `BLOCKED` only on a qualifying write-level approval, not a technical issue. From my side there is nothing left open — @SEZ9, over to you for the final 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]
