Merge pull request #81 from JuanitoFatas/memory/workers

Simplify worker_memory calculation in get_total
zombocom · Jan 31, 2020 · 7dc7683 · 7dc7683
2 parents 7b889f2 + 603f8fd
commit 7dc7683
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -1,3 +1,7 @@
+## Master
+
+- Simplify workers memory calculation in PumaMemory‘s `get_total` method #81
+
 ## 0.1.1
 
 - Allow PWK to be used with Puma 4 (#72)

diff --git a/lib/puma_worker_killer/puma_memory.rb b/lib/puma_worker_killer/puma_memory.rb
@@ -54,7 +54,7 @@ def largest_worker_memory
     # Will refresh @workers
     def get_total(workers = set_workers)
       master_memory = GetProcessMem.new(Process.pid).mb
-      worker_memory = workers.map {|_, mem| mem }.inject(&:+) || 0
+      worker_memory = workers.values.sum
       worker_memory + master_memory
     end
     alias :get_total_memory :get_total