concord-server: fix PolicyCache reloading loop

walmartlabs · Jan 14, 2025 · 8265025 · 8265025
1 parent 2f6f43a
commit 8265025
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/server/impl/src/main/java/com/walmartlabs/concord/server/policy/PolicyCache.java b/server/impl/src/main/java/com/walmartlabs/concord/server/policy/PolicyCache.java
@@ -46,6 +46,7 @@
 
 import static com.walmartlabs.concord.server.jooq.Tables.POLICIES;
 import static com.walmartlabs.concord.server.jooq.Tables.POLICY_LINKS;
+import static java.util.concurrent.TimeUnit.MILLISECONDS;
 
 public class PolicyCache implements BackgroundTask {
 
@@ -176,7 +177,9 @@ private void run() {
                     if (lastRefreshRequestAt > now) {
                         lastRefreshRequestAt = now;
                     } else {
-                        refreshMutex.wait(cacheCfg.getReloadInterval().toMillis());
+                        //noinspection ResultOfMethodCallIgnored
+                        refreshMutex.newCondition()
+                                .await(cacheCfg.getReloadInterval().toMillis(), MILLISECONDS);
                     }
                 } finally {
                     refreshMutex.unlock();