DanielLeens commented on PR #10808: URL: https://github.com/apache/seatunnel/pull/10808#issuecomment-4968421977
Rechecked the same unchanged head `4b6316e5b412` after the CI / merge-gate fact changed. ## Runtime path rechecked ```text That means when Hazelcast mastership is temporarily pointing at a lite worker and there is no coordinator-capable member visible in the current membership snapshot, the method falls back to the lite worker address instead of returning `null`. # Why this is still blocking ``` ## Merge conclusion ### Conclusion: still needs fixes before merge 1. Blocking items - The source-side blockers from my previous Daniel review still stand on this unchanged head. 2. Current gate update - The CI / merge-gate fact I referenced earlier has changed. The current status is `PENDING gate: Build`: https://github.com/apache/seatunnel/runs/86906826206 So my technical conclusion is unchanged on merge readiness: this PR still needs the earlier code-side fixes first. -- 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]
