DanielLeens commented on PR #12862: URL: https://github.com/apache/seatunnel/pull/12862#issuecomment-6574866039
Quick status update for head `1ce4e26cdbdcac4cf2d901c807463fe82cdf2822`, which is unchanged since the last discussion. The `Build` check on this head has now completed with `SUCCESS` (finished 2026-11-02T05:09Z): https://github.com/apache/seatunnel/runs/125043794701 . That clears the `unit-test (17, ubuntu-latest)` / `PhysicalVertexTest` startup failure that was blocking earlier, so the rerun suggested above did the job. I did not independently attribute the earlier port-binding failure; the green result on the same SHA is consistent with it having been environmental, and no code change was needed. My source-side position is unchanged: the Spark decimal changes are clear from my side, and I am not asking for any code change. The PR currently shows `MERGEABLE` with `REVIEW_REQUIRED`, so the only remaining step is a write-capable committer's approval and queue action (my approval does not satisfy the branch protection; I only have read access). @davidzollo, could you please take a look when you have a moment? @zhengxiang378928908-code, please keep the branch untouched until it goes through the queue so the approval and the Build result stay bound to the same head. This is based on reading the code and GitHub's check state; I did not run anything locally. -- 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]
