commit 5af58467ecd4a0f7646269f0bdbccd57f21b5df8
author Guillaume Sauthier <guillaume@apache.org> Wed Oct 02 14:56:38 2013 +0000
committer Guillaume Sauthier <guillaume@apache.org> Wed Oct 02 14:56:38 2013 +0000
tree 53229b1a28a96a7968290763bc1e3f508f019509
parent e4aea0bbc9fb0c3df81e548a54edc0d5ee5f3fce

Quick fix: executionTime counter was still running after exception

git-svn-id: https://svn.apache.org/repos/asf/felix/trunk@1528507 13f79535-47bb-0310-9956-ffa450edef68

ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/extender/internal/queue/JobInfoCallable.java

diff --git a/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/extender/internal/queue/JobInfoCallable.java b/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/extender/internal/queue/JobInfoCallable.java
index 8ad2e1c..2216533 100644
--- a/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/extender/internal/queue/JobInfoCallable.java
+++ b/ipojo/runtime/core/src/main/java/org/apache/felix/ipojo/extender/internal/queue/JobInfoCallable.java
@@ -112,8 +112,10 @@
         try {
             m_queueNotifier.fireStartedJobInfo(this);
             result = m_delegate.call();
+            endTime = System.currentTimeMillis();
             return result;
         } catch (Exception e) {
+            endTime = System.currentTimeMillis();
             m_queueNotifier.fireFailedJobInfo(this, e);
             if (m_callback != null) {
                 m_callback.error(this, e);
@@ -123,7 +125,6 @@
         } finally {
             m_statistic.getCurrentsCounter().decrementAndGet();
             m_statistic.getFinishedCounter().incrementAndGet();
-            endTime = System.currentTimeMillis();
 
             // Only exec success callbacks when no error occurred
             if (exception == null) {