[GitHub] activemq-artemis pull request #1502: [ARTEMIS-1382] Improve artemis-server l...

[treblereel]

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

    https://github.com/apache/activemq-artemis/pull/1502#discussion_r137271018
  
    --- Diff: 
artemis-server/src/main/java/org/apache/activemq/artemis/core/server/cluster/ClusterManager.java
 ---
    @@ -542,15 +542,15 @@ public void clear() {
              try {
                 bridge.stop();
              } catch (Exception e) {
    -            ActiveMQServerLogger.LOGGER.warn(e.getMessage(), e);
    --- End diff --
    
    yeap, but here we have 3 options howto log it 
    * logger.warn(e.getMessage(), e);
    * ActiveMQServerLogger.LOGGER.warn(e.getMessage(), e);
    * ActiveMQServerLogger.LOGGER.failedToStopBridge(e);
    What is the best in this case and why ?



---