You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@druid.apache.org by ni...@apache.org on 2019/06/27 16:51:43 UTC

[incubator-druid] branch master updated: InterruptedException is never thrown (#7963)

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

nishant pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-druid.git


The following commit(s) were added to refs/heads/master by this push:
     new fba91b7  InterruptedException is never thrown (#7963)
fba91b7 is described below

commit fba91b77d0f29a93af15b670bbc7ab4b610c699b
Author: Fokko Driesprong <fo...@apache.org>
AuthorDate: Thu Jun 27 18:51:36 2019 +0200

    InterruptedException is never thrown (#7963)
    
    And therefore the block will always be false
---
 codestyle/spotbugs-exclude.xml                                         | 1 -
 .../org/apache/druid/emitter/ambari/metrics/AmbariMetricsEmitter.java  | 3 ---
 2 files changed, 4 deletions(-)

diff --git a/codestyle/spotbugs-exclude.xml b/codestyle/spotbugs-exclude.xml
index 5db5945..fbaec91 100644
--- a/codestyle/spotbugs-exclude.xml
+++ b/codestyle/spotbugs-exclude.xml
@@ -29,7 +29,6 @@
 -->
 <FindBugsFilter>
     <Bug pattern="AT_OPERATION_SEQUENCE_ON_CONCURRENT_ABSTRACTION"/>
-    <Bug pattern="BC_IMPOSSIBLE_INSTANCEOF"/>
     <Bug pattern="BC_UNCONFIRMED_CAST"/>
     <Bug pattern="BIT_SIGNED_CHECK_HIGH_BIT"/>
     <Bug pattern="BX_UNBOXING_IMMEDIATELY_REBOXED"/>
diff --git a/extensions-contrib/ambari-metrics-emitter/src/main/java/org/apache/druid/emitter/ambari/metrics/AmbariMetricsEmitter.java b/extensions-contrib/ambari-metrics-emitter/src/main/java/org/apache/druid/emitter/ambari/metrics/AmbariMetricsEmitter.java
index 519bee4..d1ff8e3 100644
--- a/extensions-contrib/ambari-metrics-emitter/src/main/java/org/apache/druid/emitter/ambari/metrics/AmbariMetricsEmitter.java
+++ b/extensions-contrib/ambari-metrics-emitter/src/main/java/org/apache/druid/emitter/ambari/metrics/AmbariMetricsEmitter.java
@@ -193,9 +193,6 @@ public class AmbariMetricsEmitter extends AbstractTimelineMetricsSink implements
       }
       catch (Exception e) {
         log.error(e, e.getMessage());
-        if (e instanceof InterruptedException) {
-          Thread.currentThread().interrupt();
-        }
       }
 
     }


---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@druid.apache.org
For additional commands, e-mail: commits-help@druid.apache.org