[GitHub] [helix] kaisun2000 edited a comment on pull request #1445: Make ClusterStateVerifier closable

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

kaisun2000 edited a comment 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 remove other deprecated class usage in our code base. We need to limit the scope of breaking up #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