DanielLeens commented on PR #12882:
URL: https://github.com/apache/seatunnel/pull/12882#issuecomment-6377512708

   Thanks for the update, @zhengxiang378928908-code, and for linking the run. I 
checked fork run 39915800508: it is `completed` / `success` and bound to head 
`517d260`, so it matches the current PR head. `unit-test` is green on JDK 11 
and 17 for both Ubuntu and Windows, all `updated-modules-integration-test` 
lanes are green, and the remaining jobs are `skipped` (not failed). That clears 
the earlier `JdbcOptionRuleCompletenessTest` failure.
   
   There is no new code since my last review, so my source-level conclusion is 
unchanged: no source-level blocker. I did not rerun anything locally. I also 
did not separately confirm which lane executed `JdbcDmIT`, so I am relying on 
your report for that detail.
   
   Nothing further is needed from you. The PR is still waiting on the required 
review approval, which is not something you need to act on, and @SEZ9 can 
finish the independent recheck. Thanks again for the clear validation write-up.
   


-- 
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]

Reply via email to