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/07 01:18:35 UTC

[GitHub] [helix] kaisun2000 commented on pull request #1445: Make ClusterStateVerifier closable

kaisun2000 commented on pull request #1445:
URL: https://github.com/apache/helix/pull/1445#issuecomment-704636135


   > "This class is deprecated, please use dedicated verifier classes, such as BestPossibleExternalViewVerifier, etc, in tools.ClusterVerifiers."
   > 
   > Is it possible that we just stop using this class?
   
   @jiajunwang, theoretically, you are right. On the other hand, we did not really remove deprecated class usage in other place of the code.  Let us limit the scope of breaking down #1227. 
   How about let create a separate issue to track "removing usage of deprecated ClusterVerifier"


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