curcur commented on pull request #16653:
URL: https://github.com/apache/flink/pull/16653#issuecomment-892379997


   @akalash 
   
   I've looked through the code again, I think we should probably also keep 
   ```      
           timersFinishedFuture.get();
           systemTimersFinishedFuture.get();
   ```
   
   in the task thread for the safety purposes. 
   
   I mean we move this two lines of code into the actionExecutor and keep the 
same copy in its origin place.
   
   I think for normal task, actionExecutor is the same task thread, but there 
is also some cases (like sources) that they maybe  different 


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