From 2c07ca2e09297057c0264dcdd5c9f32602cddd64 Mon Sep 17 00:00:00 2001 From: Amnon Heiman Date: Tue, 17 May 2016 11:26:18 +0300 Subject: [PATCH] LatencyMetrics: Move to APITimer The APITimer uses a different endpoint not to break existing API. The addNano functionality was removed as all of the values are updated from the APi. Signed-off-by: Amnon Heiman --- .../org/apache/cassandra/metrics/LatencyMetrics.java | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/src/main/java/org/apache/cassandra/metrics/LatencyMetrics.java b/src/main/java/org/apache/cassandra/metrics/LatencyMetrics.java index 2d100f4..1f4a9b4 100644 --- a/src/main/java/org/apache/cassandra/metrics/LatencyMetrics.java +++ b/src/main/java/org/apache/cassandra/metrics/LatencyMetrics.java @@ -108,7 +108,7 @@ public class LatencyMetrics { this.namePrefix = namePrefix; paramName = (paramName == null)? "" : "/" + paramName; - latency = APIMetrics.newTimer(url + "/histogram" + paramName, + latency = APIMetrics.newTimer(url + "/moving_average_histogram" + paramName, factory.createMetricName(namePrefix + "Latency"), TimeUnit.MICROSECONDS, TimeUnit.SECONDS); totalLatency = APIMetrics.newCounter(url + paramName, @@ -135,12 +135,7 @@ public class LatencyMetrics { /** takes nanoseconds **/ public void addNano(long nanos) { - // convert to microseconds. 1 millionth - latency.update(nanos, TimeUnit.NANOSECONDS); - totalLatency.inc(nanos / 1000); - for (LatencyMetrics parent : parents) { - parent.addNano(nanos); - } + // the object is only updated from the API } public void release() {