From 916aead9ab081c7bb6ab6bce39a0d643ebcc3104 Mon Sep 17 00:00:00 2001 From: wilderrodrigues Date: Thu, 25 Jun 2015 08:09:18 +0200 Subject: [PATCH] Using Profiler class, from the utils package, instead of System.currentTimeMillis() Signed-off-by: wilderrodrigues This closes #525 --- .../com/cloud/cluster/ClusterServiceServletImpl.java | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/framework/cluster/src/com/cloud/cluster/ClusterServiceServletImpl.java b/framework/cluster/src/com/cloud/cluster/ClusterServiceServletImpl.java index 250b88604c4..9b3472447e9 100644 --- a/framework/cluster/src/com/cloud/cluster/ClusterServiceServletImpl.java +++ b/framework/cluster/src/com/cloud/cluster/ClusterServiceServletImpl.java @@ -27,6 +27,8 @@ import org.apache.commons.httpclient.methods.PostMethod; import org.apache.commons.httpclient.params.HttpClientParams; import org.apache.log4j.Logger; +import com.cloud.utils.Profiler; + public class ClusterServiceServletImpl implements ClusterService { private static final long serialVersionUID = 4574025200012566153L; private static final Logger s_logger = Logger.getLogger(ClusterServiceServletImpl.class); @@ -87,16 +89,19 @@ public class ClusterServiceServletImpl implements ClusterService { int response = 0; String result = null; try { - final long startTick = System.currentTimeMillis(); + final Profiler profiler = new Profiler(); + profiler.start(); response = client.executeMethod(method); if (response == HttpStatus.SC_OK) { result = method.getResponseBodyAsString(); + profiler.stop(); if (s_logger.isDebugEnabled()) { - s_logger.debug("POST " + _serviceUrl + " response :" + result + ", responding time: " + (System.currentTimeMillis() - startTick) + " ms"); + s_logger.debug("POST " + _serviceUrl + " response :" + result + ", responding time: " + profiler.getDurationInMillis() + " ms"); } } else { + profiler.stop(); s_logger.error("Invalid response code : " + response + ", from : " + _serviceUrl + ", method : " + method.getParameter("method") + " responding time: " + - (System.currentTimeMillis() - startTick)); + profiler.getDurationInMillis()); } } catch (final HttpException e) { s_logger.error("HttpException from : " + _serviceUrl + ", method : " + method.getParameter("method"));