You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@dolphinscheduler.apache.org by GitBox <gi...@apache.org> on 2021/06/02 14:06:11 UTC

[GitHub] [dolphinscheduler] blackberrier commented on pull request #5572: [Improvement-5539][Master] Check status of taskInstance from cache

blackberrier commented on pull request #5572:
URL: https://github.com/apache/dolphinscheduler/pull/5572#issuecomment-853057809


   @ruanwenjun 
   Hi, I have updated my code.
   1. use timer and scheduleAtFixedRate instead of while(stopper) and thread.sleep. And add close method as well.
   2. add synchronized code segment to handler concurrent modification
   3. remove useless code
   
   Can you see the latest code? Review it please, Thanks!


-- 
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:
users@infra.apache.org