pnowojski commented on pull request #11541:
URL: https://github.com/apache/flink/pull/11541#issuecomment-638644705


   There are a couple of things left to do there. 
   
   1. For example test is marked with Ignore and I’m not sure if that’s a 
mistake, as the commit marking it ignored seemed to be fixing something. But it 
was 5 years ago so who knows… removing Ignore might introduce test instability, 
so it would have to be thoroughly checked.
   2. There are rebase conflicts.
   3. I think in your test you should be joining the spawned threads and 
checking their’s exceptions.
   
   If you prefer we can take over those issues PR and merge it in a couple of 
weeks after release 1.11 testing is done.


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