From 3ecb47da5a9fb594ff787680c1420e40a314aff4 Mon Sep 17 00:00:00 2001
From: Athou <jeremiepanzer@gmail.com>
Date: Sun, 18 Aug 2013 16:42:01 +0200
Subject: [PATCH] use timers instead of meters

---
 .../frontend/rest/resources/AbstractREST.java    | 16 +++++++++-------
 1 file changed, 9 insertions(+), 7 deletions(-)

diff --git a/src/main/java/com/commafeed/frontend/rest/resources/AbstractREST.java b/src/main/java/com/commafeed/frontend/rest/resources/AbstractREST.java
index 929e1a3d..88f6ca55 100644
--- a/src/main/java/com/commafeed/frontend/rest/resources/AbstractREST.java
+++ b/src/main/java/com/commafeed/frontend/rest/resources/AbstractREST.java
@@ -24,7 +24,6 @@ import org.apache.wicket.protocol.http.servlet.ServletWebResponse;
 import org.apache.wicket.request.cycle.RequestCycle;
 import org.apache.wicket.util.crypt.Base64;
 
-import com.codahale.metrics.Meter;
 import com.codahale.metrics.MetricRegistry;
 import com.commafeed.backend.dao.UserDAO;
 import com.commafeed.backend.model.User;
@@ -100,7 +99,6 @@ public abstract class AbstractREST {
 	@AroundInvoke
 	public Object intercept(InvocationContext context) throws Exception {
 		Method method = context.getMethod();
-		handleMetric(method);
 
 		// check security
 		boolean allowed = true;
@@ -127,12 +125,16 @@ public abstract class AbstractREST {
 
 		}
 
-		return context.proceed();
-	}
+		Object result = null;
+		com.codahale.metrics.Timer.Context timer = metrics.timer(
+				MetricRegistry.name(method.getDeclaringClass(), method.getName(), "responseTime")).time();
+		try {
+			result = context.proceed();
+		} finally {
+			timer.stop();
+		}
 
-	private void handleMetric(Method method) {
-		Meter callCount = metrics.meter(MetricRegistry.name(method.getDeclaringClass(), method.getName(), "callCount"));
-		callCount.mark();
+		return result;
 	}
 
 	private boolean checkRole(Role requiredRole) {