You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@helix.apache.org by GitBox <gi...@apache.org> on 2022/03/18 06:18:37 UTC

[GitHub] [helix] Jackie-Jiang opened a new issue #1990: Race condition between instance drop and state transition message creation

Jackie-Jiang opened a new issue #1990:
URL: https://github.com/apache/helix/issues/1990


   ### Describe the bug
   There is a potential race condition when an instance is dropped and a state transition message is created at the same time. The state transition message can recreate the `INSTANCE/<instanceId>` ZK directory, and cause instance show up in the cluster again, but without instance config.
   
   ### To Reproduce
   Remove an instance from the IdealState and drop it at the same time.
   
   ### Expected behavior
   The state transition message should not create the ZK directory if it does not exist.
   


-- 
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: reviews-unsubscribe@helix.apache.org

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


[GitHub] [helix] junkaixue commented on issue #1990: Race condition between instance drop and state transition message creation

Posted by GitBox <gi...@apache.org>.
junkaixue commented on issue #1990:
URL: https://github.com/apache/helix/issues/1990#issuecomment-1074291936


   Thanks @Jackie-Jiang. Yeah, this is an known issue that ZK set will automatically create the folder if it does not exists. We may fix it with updater for history update.


-- 
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: reviews-unsubscribe@helix.apache.org

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


[GitHub] [helix] junkaixue edited a comment on issue #1990: Race condition between instance drop and state transition message creation

Posted by GitBox <gi...@apache.org>.
junkaixue edited a comment on issue #1990:
URL: https://github.com/apache/helix/issues/1990#issuecomment-1074291936


   Thanks @Jackie-Jiang. Yeah, this is an known issue that ZK set will automatically create the folder if it does not exists. We may fix it with updater for history update.
   
   We will work on it soon.


-- 
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: reviews-unsubscribe@helix.apache.org

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