Merge pull request quasor#3 from redterror/master

Move the 'mean' calculation to before the pct_threshold adjustments
adamgibbins · Apr 27, 2012 · ca10606 · ca10606
2 parents 9d1c55e + 1d726bd
commit ca10606
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/statsd/graphite.rb b/lib/statsd/graphite.rb
@@ -55,14 +55,14 @@ def flush_stats
             max_at_threshold = max
 
             if (count > 1)
+              # average all the timing data
+              sum = values.inject( 0 ) { |s,x| s+x }
+              mean = sum / values.count
+
               # strip off the top 100-threshold
               threshold_index = (((100 - pct_threshold) / 100.0) * count).round
               values = values[0..-threshold_index]
               max_at_threshold = values.last
-
-              # average the remaining timings
-              sum = values.inject( 0 ) { |s,x| s+x }
-              mean = sum / values.count
             end
 
             message = ""