Merge pull request #83 from DataDog/jaime/instancefail

[jmx] if the exception is an IOException, throw it. Instance should be removed.
DataDog · Mar 22, 2016 · b957b28 · b957b28
2 parents 2c485f0 + 6cb005b
commit b957b28
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 0 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -6,3 +6,6 @@ jdk:
 notifications:
   slack:
     secure: bUPdpDtrBYvlJ93AmfqvPU6ZSZhFHZJiIoNf0bXJVNZyxp2rjfMC8BG6XkesbKk62s7NJM0yfZhhLAzjNG3tSA9iL5i3npbl8kRaUi2zEDOCEG0zZ+FX2NxTiNH937S6QRmCBHe9/KEkSRuMY3HNRUidAq8w6dVCrWxxj5NwxjU=
+
+addons:
+  hostname: dd-jmxfetch-testhost
diff --git a/src/main/java/org/datadog/jmxfetch/Instance.java b/src/main/java/org/datadog/jmxfetch/Instance.java
@@ -166,6 +166,8 @@ public LinkedList<HashMap<String, Object>> getMetrics() throws IOException {
                 if (this.failingAttributes.contains(jmxAttr)) {
                     this.failingAttributes.remove(jmxAttr);
                 }
+            } catch (IOException e) {
+                throw e;
             } catch (Exception e) {
                 LOGGER.debug("Cannot get metrics for attribute: " + jmxAttr, e);
                 if (this.failingAttributes.contains(jmxAttr)) {