DanielLeens commented on PR #12824: URL: https://github.com/apache/seatunnel/pull/12824#issuecomment-6227285650
Quick follow-up to correct the CI status in my previous review: the `Build` check on the current head (`d63c675`) has now completed successfully, so the "Build still in progress" note no longer applies. One new item on the merge state: GitHub now reports merge conflicts between this branch and the latest `dev` (the branch is 7 commits behind). Could you please sync with the latest `dev` and resolve the conflicts first? Once that is pushed and the checks have rerun, I'm happy to take another look. The remaining non-blocking notes from my earlier review (Issues 1-3) are unchanged. Thanks for the contribution! -- 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]
