[GitHub] [helix] desaikomal commented on a diff in pull request #2505: Replace deprecated verifier with new set of Verifiers

["desaikomal (via GitHub)" <gi...@apache.org>]

desaikomal commented on code in PR #2505:
URL: https://github.com/apache/helix/pull/2505#discussion_r1204589618


##########
helix-core/src/test/java/org/apache/helix/integration/manager/TestDistributedControllerManager.java:
##########
@@ -71,18 +72,18 @@ public void simpleIntegrationTest() throws Exception {
           new MockMSModelFactory());
       distributedControllers[i].connect();
     }
+    BestPossibleExternalViewVerifier verifier = new BestPossibleExternalViewVerifier
+        .Builder(clusterName).setZkAddress(ZK_ADDR).build();
 
-    boolean result = ClusterStateVerifier
-        .verifyByZkCallback(new BestPossAndExtViewZkVerifier(ZK_ADDR, clusterName));
+    boolean result = verifier.verifyByZkCallback();
     Assert.assertTrue(result);
 
     // disconnect first distributed-controller, and verify second takes leadership
     distributedControllers[0].disconnect();
 
     // verify leader changes to localhost_12919
     Thread.sleep(100);

Review Comment:
   since I got +1, i will do it. Typically, i try to do specific change rather than mix 2/3 things in one. cognitive load increases.



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