Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

skip setting intermediate variable #208

Merged
merged 1 commit into from

2 participants

@draco2003

If we aren't going to pull this logic out pending the response to pull #207 we probably don't need to do a double hop through the valuePerSecond variable.

@mrtazz
Owner

good catch, thanks for the patch!

@mrtazz mrtazz merged commit 641e4ac into etsy:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Dec 12, 2012
  1. @draco2003
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 2 deletions.
  1. +1 −2  lib/process_metrics.js
View
3  lib/process_metrics.js
@@ -12,8 +12,7 @@ var process_metrics = function (metrics, flushInterval, ts, flushCallback) {
var value = counters[key];
// calculate "per second" rate
- var valuePerSecond = value / (flushInterval / 1000);
- counter_rates[key] = valuePerSecond;
+ counter_rates[key] = value / (flushInterval / 1000);
}
for (key in timers) {
Something went wrong with that request. Please try again.