StephanEwen opened a new pull request #13593:
URL: https://github.com/apache/flink/pull/13593


   ## What is the purpose of the change
   
   Fixes two test instabilities on the `SplitFetcherTest`.
   
   ## Brief change log
   
   The test logic was previously incorrect. This now adjusts the logic to 
correctly reflect the requirements for idleness notifications, as required by 
the reader, where at least one of the following conditions must be true:
   
     - Either the fetcher was already set to idle when the reader thread pulled 
the last fetch from the queue, because that is when the reader thread checks 
for fetcher being idle and shuts it down (which signals end of input)
     - Or the fetcher was not yet marked idle, because the fetch was pulled 
before that flag was set. In that case, the availability future must be 
completed, so that the reader thread goes back to checking the fetcher.


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to