davidzollo commented on PR #11727:
URL: https://github.com/apache/seatunnel/pull/11727#issuecomment-5739948024

   > @DanielLeens — pushed the F7 breadcrumb in 
[06c2ef0](https://github.com/apache/seatunnel/commit/06c2ef03e73199d7ea35932a47385b03969b2b3f).
 The race test's class Javadoc now states explicitly that the 
redeploy-vs-taskDone half lives in deployLocalTask() and the tracker teardown, 
that neither is touched here, and that it's #12238's design and tracked against 
it — so nothing in this test reads as covering it.
   > 
   > Thanks for re-checking the diff independently rather than taking my grep 
on faith.
   > 
   > @SEZ9 — that closes the F7 coverage thread from my side. F2 is the one 
item left for you to confirm against the diff.
   
   +1
   LGTM if CI passes
   
   This PR is about 40 commits behind the latest code. I merge
    the upstream code to make it better to get through CI  check.


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