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 2020/10/08 06:00:07 UTC
[GitHub] [helix] kaisun2000 commented on pull request #1450: HelixClusterVerifier verify() with default waitTillVerify time -- part two
kaisun2000 commented on pull request #1450:
URL: https://github.com/apache/helix/pull/1450#issuecomment-705349006
This PR is approved, please help to merge into trunk.
> HelixClusterVerifier verify() and related method may return
pre-maturely. The reason is that the verify the converging stable
condition too early before controller has a chance to make
change. Basically the previous stable state is mistaken as the
expected next stable state. Part two.
----------------------------------------------------------------
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