lostluck commented on PR #35337:
URL: https://github.com/apache/beam/pull/35337#issuecomment-2981636347

   OH I  see what's happening here now.
   
   The problem is post-loop *no longer checks for Bundle termination correctly*.
   
   This is due to some re-arrangements made a long time ago, when the progress 
loop and the data checking were more independent. Now they're done all in the 
loop, and then we don't check afterwards.
   
   ---------
   
   The fix is aborting progress + splits differently, but still wait for the 
normal completion signals.
   
   Putting specific comments now.
   


-- 
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: github-unsubscr...@beam.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to