Archives/Athou_commafeed
1
0
Fork 1
mirror of https://github.com/Athou/commafeed.git synced 2026-03-21 21:37:29 +00:00
Code Issues Packages Projects Releases Wiki Activity

use timers instead of meters

Browse Source
This commit is contained in:
Athou
2013-08-18 16:42:01 +02:00
parent ae03b42c6d
commit 3ecb47da5a
1 changed files with 9 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

16
src/main/java/com/commafeed/frontend/rest/resources/AbstractREST.java
View File

@@ -24,7 +24,6 @@ import org.apache.wicket.protocol.http.servlet.ServletWebResponse;
import org.apache.wicket.request.cycle.RequestCycle; import org.apache.wicket.request.cycle.RequestCycle;
import org.apache.wicket.util.crypt.Base64; import org.apache.wicket.util.crypt.Base64;
import com.codahale.metrics.Meter;
import com.codahale.metrics.MetricRegistry; import com.codahale.metrics.MetricRegistry;
import com.commafeed.backend.dao.UserDAO; import com.commafeed.backend.dao.UserDAO;
import com.commafeed.backend.model.User; import com.commafeed.backend.model.User;
@@ -100,7 +99,6 @@ public abstract class AbstractREST {
@AroundInvoke @AroundInvoke
public Object intercept(InvocationContext context) throws Exception { public Object intercept(InvocationContext context) throws Exception {
Method method = context.getMethod(); Method method = context.getMethod();
handleMetric(method);
// check security // check security
boolean allowed = true; 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) { return result;
Meter callCount = metrics.meter(MetricRegistry.name(method.getDeclaringClass(), method.getName(), "callCount"));
callCount.mark();
} }
private boolean checkRole(Role requiredRole) { private boolean checkRole(Role requiredRole) {