You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@sling.apache.org by ro...@apache.org on 2020/04/17 12:32:07 UTC

[sling-org-apache-sling-commons-metrics-rrd4j] 01/02: SLING-9382 Shutdown executor before closing db connection

This is an automated email from the ASF dual-hosted git repository.

rombert pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-commons-metrics-rrd4j.git

commit 7ba701633001dade0f541370341d8a93c6ac4f65
Author: Harsh Mehta <hm...@adobe.com>
AuthorDate: Fri Apr 17 17:00:43 2020 +0530

    SLING-9382 Shutdown executor before closing db connection
---
 .../java/org/apache/sling/commons/metrics/rrd4j/impl/RRD4JReporter.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/main/java/org/apache/sling/commons/metrics/rrd4j/impl/RRD4JReporter.java b/src/main/java/org/apache/sling/commons/metrics/rrd4j/impl/RRD4JReporter.java
index 5cb9418..e9c1063 100644
--- a/src/main/java/org/apache/sling/commons/metrics/rrd4j/impl/RRD4JReporter.java
+++ b/src/main/java/org/apache/sling/commons/metrics/rrd4j/impl/RRD4JReporter.java
@@ -210,6 +210,7 @@ class RRD4JReporter extends ScheduledReporter {
 
     @Override
     public void close() {
+    	super.close();
         try {
             // write an unknown sample before closing the DB
             if (!rrdDB.isClosed()) {
@@ -219,7 +220,6 @@ class RRD4JReporter extends ScheduledReporter {
         } catch (IOException e) {
             LOGGER.warn("Closing RRD failed", e);
         }
-        super.close();
     }
 
     @Override