DanielLeens commented on PR #11892: URL: https://github.com/apache/seatunnel/pull/11892#issuecomment-6682774644
Thanks @JacobZheng0927. A quick gate update on the unchanged head `7c0b24465d`: CI is still green (the apache [Build](https://github.com/apache/seatunnel/runs/126428862204) check now reports success, matching the fork run cited in my last review), but one fact from that review has changed. GitHub now reports a merge conflict with `dev`, so my earlier note that no sync was needed no longer holds. Suggested next step: when you push the fix for Issue 1 (Greenplum native-driver binding, with the loader-level test and the docs sentence), please also merge the latest `dev` and resolve the conflict in the same update. The new head will need fresh checks and a re-review. My source conclusion is otherwise unchanged. -- 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]
