SEZ9 commented on PR #12208: URL: https://github.com/apache/seatunnel/pull/12208#issuecomment-6487248541
Hi @88fantasy, a brief note so this doesn't get lost: nothing new from my side beyond my last two comments. Once you have re-synced the branch with `dev` and pushed, please share the new head so I can re-check the gate on it. If the [unit-test (11, ubuntu-latest)](https://github.com/88fantasy/seatunnel/actions/runs/40477948026/job/122104731061) job still fails on the synced head, the failing test name(s) in `seatunnel-engine-server` would help; if it goes green, that is sufficient. The source-level points from my earlier review remain resolved. Thanks! <!-- streview-comment:2120 --> -- 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]
