This is an automated email from the ASF dual-hosted git repository.

technoboy pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/pulsar.git


The following commit(s) were added to refs/heads/master by this push:
     new 7b93e1a051b [cleanup][test] remove redundant code (#18487)
7b93e1a051b is described below

commit 7b93e1a051b43d3f7527439b9a00a02303bb0d31
Author: labuladong <[email protected]>
AuthorDate: Fri Nov 18 11:48:13 2022 +0800

    [cleanup][test] remove redundant code (#18487)
---
 .../java/org/apache/pulsar/broker/loadbalance/LoadBalancerTest.java    | 3 ---
 1 file changed, 3 deletions(-)

diff --git 
a/pulsar-broker/src/test/java/org/apache/pulsar/broker/loadbalance/LoadBalancerTest.java
 
b/pulsar-broker/src/test/java/org/apache/pulsar/broker/loadbalance/LoadBalancerTest.java
index e60a873ca5a..362b3e0cfcf 100644
--- 
a/pulsar-broker/src/test/java/org/apache/pulsar/broker/loadbalance/LoadBalancerTest.java
+++ 
b/pulsar-broker/src/test/java/org/apache/pulsar/broker/loadbalance/LoadBalancerTest.java
@@ -164,7 +164,6 @@ public class LoadBalancerTest {
     }
 
     private void loopUntilLeaderChangesForAllBroker(List<PulsarService> 
activePulsars, LeaderBroker oldLeader) {
-        int loopCount = 0;
         Awaitility.await()
             .pollInterval(1, TimeUnit.SECONDS)
             .atMost(MAX_RETRIES, TimeUnit.SECONDS)
@@ -181,8 +180,6 @@ public class LoadBalancerTest {
                 }
                 return settled;
             });
-        // Check if maximum retries are already done. If yes, assert.
-        Assert.assertNotEquals(loopCount, MAX_RETRIES, "Leader is not changed 
even after maximum retries.");
     }
 
     /*

Reply via email to