Avoid checking if all workers reached timeout unless idle timeout is …

…configured (puma#3341)

Puma::Cluster#run constantly checks if all workers have reached their idle timeout even when the feature isn't being used. This change ensures that this check only runs when idle_timeout is configured.

Also includes a micro-optimisation which ensures the idle workers state hash isn't unnecessarily initialised until actually required (either a worker indicates timeout reached via pipe or Puma::Cluster#run checks if all_workers_idle_timed_out?).

All of these changes are covered by existing tests.

lib/puma/cluster.rb

@@ -348,8 +348,6 @@ def setup_signals
     def run
       @status = :run
 
-      @idle_workers = {}
-
       output_header "cluster"
 
       # This is aligned with the output from Runner, see Runner#output_header
@@ -440,7 +438,7 @@ def run
 
         while @status == :run
           begin
-            if all_workers_idle_timed_out?
+            if @options[:idle_timeout] && all_workers_idle_timed_out?
               log "- All workers reached idle timeout"
               break
             end
@@ -500,10 +498,10 @@ def run
                     booted = true
                   end
                 when Puma::Const::PipeRequest::IDLE
-                  if @idle_workers[pid]
-                    @idle_workers.delete pid
+                  if idle_workers[pid]
+                    idle_workers.delete pid
                   else
-                    @idle_workers[pid] = true
+                    idle_workers[pid] = true
                   end
                 end
               else
@@ -608,7 +606,11 @@ def timeout_workers
     end
 
     def idle_timed_out_worker_pids
-      @idle_workers.keys
+      idle_workers.keys
+    end
+
+    def idle_workers
+      @idle_workers ||= {}
     end
   end
 end