You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by to...@apache.org on 2016/11/28 15:01:29 UTC

svn commit: r1771756 - /sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/monitor/impl/MonitoringDistributionPackageBuilder.java

Author: tommaso
Date: Mon Nov 28 15:01:29 2016
New Revision: 1771756

URL: http://svn.apache.org/viewvc?rev=1771756&view=rev
Log:
SLING-6324 - removed duplicated / unneeded mbean registrations

Modified:
    sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/monitor/impl/MonitoringDistributionPackageBuilder.java

Modified: sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/monitor/impl/MonitoringDistributionPackageBuilder.java
URL: http://svn.apache.org/viewvc/sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/monitor/impl/MonitoringDistributionPackageBuilder.java?rev=1771756&r1=1771755&r2=1771756&view=diff
==============================================================================
--- sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/monitor/impl/MonitoringDistributionPackageBuilder.java (original)
+++ sling/trunk/contrib/extensions/distribution/core/src/main/java/org/apache/sling/distribution/monitor/impl/MonitoringDistributionPackageBuilder.java Mon Nov 28 15:01:29 2016
@@ -68,26 +68,17 @@ public final class MonitoringDistributio
 
     @Override
     public DistributionPackage readPackage(ResourceResolver resourceResolver, InputStream stream) throws DistributionException {
-        long start = System.currentTimeMillis();
-        DistributionPackage distributionPackage = wrapped.readPackage(resourceResolver, stream);
-        registerDistributionPackageMBean(start, distributionPackage);
-        return distributionPackage;
+        return wrapped.readPackage(resourceResolver, stream);
     }
 
     @Override
     public DistributionPackage getPackage(ResourceResolver resourceResolver, String id) throws DistributionException {
-        long start = System.currentTimeMillis();
-        DistributionPackage distributionPackage = wrapped.getPackage(resourceResolver, id);
-        registerDistributionPackageMBean(start, distributionPackage);
-        return distributionPackage;
+        return wrapped.getPackage(resourceResolver, id);
     }
 
     @Override
     public boolean installPackage(ResourceResolver resourceResolver, DistributionPackage distributionPackage) throws DistributionException {
-        long start = System.currentTimeMillis();
-        boolean installed = wrapped.installPackage(resourceResolver, distributionPackage);
-        registerDistributionPackageMBean(start, distributionPackage);
-        return installed;
+        return wrapped.installPackage(resourceResolver, distributionPackage);
     }
 
     @Override