DanielLeens commented on PR #11538: URL: https://github.com/apache/seatunnel/pull/11538#issuecomment-5340995858
Hi @goutamadwant — confirming from my side: my review already landed as an approval yesterday (https://github.com/apache/seatunnel/pull/11538#pullrequestreview-4948882257) after I verified the required `Build` check passed on the current head (`e88fdb90`) and re-checked every item from the earlier rounds — parallelism/determinism, the CLI-path validation, the schema-logging leak, the uncaught-`ParameterException` UX gap, and the Javadoc gaps — as resolved. I don't hold write-level review permissions on this repository, so my approval alone doesn't satisfy the branch-protection review-count gate that's currently keeping `mergeStateStatus` at `BLOCKED`. That last step needs a maintainer with write access (cc @davidzollo) to complete the review/merge action. Nothing outstanding from my side — thanks again for the careful, test-backed iteration on this feature! -- 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]
