From 53bb0472dd529a08e3696d5bf14c49f186b2b1a6 Mon Sep 17 00:00:00 2001
From: Demogorgon314 <kwang@streamnative.io>
Date: Mon, 21 Mar 2022 20:31:17 +0800
Subject: [PATCH] Fix wrong retry behavior in MetadataCacheTest

---
 .../test/java/org/apache/pulsar/metadata/MetadataCacheTest.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataCacheTest.java b/pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataCacheTest.java
index 1c663abd3272e..0cf2369ea6dc2 100644
--- a/pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataCacheTest.java
+++ b/pulsar-metadata/src/test/java/org/apache/pulsar/metadata/MetadataCacheTest.java
@@ -611,7 +611,7 @@ public static void assertEqualsAndRetry(Supplier<Object> actual,
     public static boolean retryStrategically(Predicate<Void> predicate, int retryCount, long intSleepTimeInMillis)
             throws Exception {
         for (int i = 0; i < retryCount; i++) {
-            if (predicate.test(null) || i == (retryCount - 1)) {
+            if (predicate.test(null)) {
                 return true;
             }
             Thread.sleep(intSleepTimeInMillis + (intSleepTimeInMillis * i));