Github user HeartSaVioR commented on a diff in the pull request:

    https://github.com/apache/storm/pull/2638#discussion_r183233652
  
    --- Diff: storm-client/src/jvm/org/apache/storm/utils/Utils.java ---
    @@ -363,7 +358,7 @@ public void run() {
                     } catch (Throwable t) {
                         if (Utils.exceptionCauseIsInstanceOf(
                                 InterruptedException.class, t)) {
    -                        LOG.info("Async loop interrupted!");
    +                        LOG.error("Async loop interrupted!", t);
    --- End diff --
    
    Here we assume that it is not an error, so if my understanding is right, 
`info` level would make more sense. (I'd second @srdo.)


---

Reply via email to