DanielLeens commented on PR #13023: URL: https://github.com/apache/seatunnel/pull/13023#issuecomment-6556451236
Thanks again. A CI fact in my existing approval on unchanged head `2ce87dd6c2b5ef0feb03ff87d92f6a707bc66036` is now outdated: the Apache Build is no longer pending; it has completed SUCCESS. The [exact-head fork Build](https://github.com/SEPURI-SAI-KRISHNA/seatunnel/actions/runs/41198287346) and its executed JDK 11/17 unit and updated-module integration jobs also succeeded. My existing technical approval stands, with the two previously identified Low recommendations remaining non-blocking. No new source review or contributor revision is requested. GitHub still reports CHANGES_REQUESTED in its overall review summary; my approval does not dismiss @nzw921rx's review. My prior full review explains why I consider the Console-versus-Assert request addressed, and the reviewer or a write-capable maintainer must reconcile that remaining effective gate. Branch divergence alone requests no sync or rerun. -- 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]
