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

   Sounds right, @merlau — that matches where I landed it too. This PR is 
already approved on `bd08491a3` and its own blocker (the offset-counter fix) is 
confirmed green in CI; the two `updated-modules-integration-test-part-5` legs 
are the only thing actually held up on #12346, since they're failing the same 
tracked dev-wide `OpengaussCDCIT` issue (#12344) that's hitting other unrelated 
PRs too, not anything in this branch.
   
   I just checked: #12346 is still open, not yet merged. Once it lands on `dev` 
and you sync this branch, a rerun should clear `part-5`, and (per my last CI 
note) `part-3`'s dependency-resolution failure and the Windows unit-test flake 
should also clear on a plain rerun without needing #12346 at all — so a fresh 
sync + rerun should get you to a fully green `Build`. Nothing further needed 
from me until then.


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