[grid] Using a more conservative execution period for session request timeout

diemol · diemol · commit cb6bc2afe413 · 2022-02-10T20:32:48.000+01:00
diff --git a/java/src/org/openqa/selenium/grid/sessionqueue/local/LocalNewSessionQueue.java b/java/src/org/openqa/selenium/grid/sessionqueue/local/LocalNewSessionQueue.java
@@ -122,8 +122,9 @@ public LocalNewSessionQueue(
     this.queue = new ConcurrentLinkedDeque<>();
     this.contexts = new ConcurrentHashMap<>();
 
-    // if retryPeriod is 0, we will schedule timeout checks every second
-    long period = this.requestTimeout.isZero() ? 1000 : this.requestTimeout.toMillis();
+    // if retryPeriod is 0, we will schedule timeout checks every 15 seconds
+    // there is no need to run this more often
+    long period = retryPeriod.isZero() ? 15000 : retryPeriod.toMillis();
     service.scheduleAtFixedRate(
       GuardedRunnable.guard(this::timeoutSessions),
       retryPeriod.toMillis(),
