[GitHub] [helix] alirezazamani commented on pull request #1131: Fix RuntimeJobDag initialization with old DAG

[GitBox <gi...@apache.org>]

alirezazamani commented on pull request #1131:
URL: https://github.com/apache/helix/pull/1131#issuecomment-652552284


   This PR is ready to be merged, approved by @dasahcc 
   
   Final commit message:
   Use Znode version to initialize RuntimeJobDag 
   
   In this commit, the znode version has been used to create the
   RuntimeJobDag in order to avoid race condition between purge
   logic and cache refresh.
   This PR specifically stabilized TestJobQueueCleanUp.
   Also this PR allows the job to continue running even if its IdealState has been removed.


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



---------------------------------------------------------------------
To unsubscribe, e-mail: reviews-unsubscribe@helix.apache.org
For additional commands, e-mail: reviews-help@helix.apache.org