Change the Loop to have a minimum wait time of 1 milli second and a maximum wait time of 1 second.

Author: willemstuursma

classes/util/Loop.php

@@ -14,6 +14,20 @@
  */
 class Loop
 {
+
+    /**
+     * Minimum time that we want to wait, between lock checks.
+     *
+     * In micro seconds.
+     */
+    private const MINIMUM_WAIT_US = 1e4;
+
+    /**
+     * Maximum time that we want to wait, between lock checks.
+     *
+     * In micro seconds.
+     */
+    private const MAXIMUM_WAIT_US = 1e6;
 
     /**
      * @var int The timeout in seconds.
@@ -69,7 +83,6 @@ public function execute(callable $code)
     {
         $this->looping = true;
 
-        $minWait = 100; // microseconds
         $deadline = microtime(true) + $this->timeout; // At this time, the lock will time out.
         $result = null;
 
@@ -79,9 +92,6 @@ public function execute(callable $code)
                 break;
             }
 
-            $min = (int) $minWait * 1.5 ** $i;
-            $max = $min * 2;
-
             /*
              * Calculate max time remaining, don't sleep any longer than that.
              */
@@ -94,9 +104,12 @@ public function execute(callable $code)
                 throw TimeoutException::create($this->timeout);
             }
 
-            $usleep = \min($usecRemaining, \random_int($min, $max));
+            $min = min((int) self::MINIMUM_WAIT_US * 1.5 ** $i, self::MAXIMUM_WAIT_US);
+            $max = min($min * 2, self::MAXIMUM_WAIT_US);
+
+            $usecToSleep = \min($usecRemaining, \random_int($min, $max));
 
-            usleep($usleep);
+            usleep($usecToSleep);
         }
 
         if (microtime(true) >= $deadline) {