activemq-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From michaelandrepearce <...@git.apache.org>
Subject [GitHub] activemq-artemis pull request #1964: [ARTEMIS-1761] Log one warning if clust...
Date Fri, 23 Mar 2018 16:42:35 GMT
Github user michaelandrepearce commented on a diff in the pull request:

    https://github.com/apache/activemq-artemis/pull/1964#discussion_r176795441
  
    --- Diff: artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/impl/BridgeImpl.java
---
    @@ -927,9 +927,7 @@ protected void connect() {
                    scheduleRetryConnectFixedTimeout(this.retryInterval);
                    return;
                 } else {
    -               if (logger.isDebugEnabled()) {
    -                  logger.debug("Bridge " + this + " is unable to connect to destination.
Retrying", e);
    -               }
    +               logger.warn("Bridge " + this + " is unable to connect to destination.
Retrying...");
    --- End diff --
    
    Not really it should have its own, it’s a different issue, and so should have its own
dedicated log error code and log output.


---

Mime
View raw message