kaisun2000 commented on pull request #1449: URL: https://github.com/apache/helix/pull/1449#issuecomment-705251143
This diff is approved. Please help to merge into trunk >fix #1448 part 1 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. We fix this issue by adding waitTillVerify() timeout in construction time of verifier. ---------------------------------------------------------------- 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: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
