Don't wrap a RuntimeException in another RuntimeException. (#488)

* Don't wrap a RuntimeException in another RuntimeException. Signed-off-by: Christian Schlichtherle <[email protected]>
prometheus · Dec 5, 2019 · ef16c43 · ef16c43
1 parent b61dd23
commit ef16c43
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/simpleclient/src/main/java/io/prometheus/client/Histogram.java b/simpleclient/src/main/java/io/prometheus/client/Histogram.java
@@ -219,6 +219,8 @@ public <E> E time(Callable<E> timeable) {
 
       try {
         return timeable.call();
+      } catch (RuntimeException e) {
+        throw e;
       } catch (Exception e) {
         throw new RuntimeException(e);
       } finally {

diff --git a/simpleclient/src/main/java/io/prometheus/client/Summary.java b/simpleclient/src/main/java/io/prometheus/client/Summary.java
@@ -226,6 +226,8 @@ public <E> E time(Callable<E> timeable) {
 
       try {
         return timeable.call();
+      } catch (RuntimeException e) {
+        throw e;
       } catch (Exception e) {
         throw new RuntimeException(e);
       } finally {