commit | eafdd346fe0d506332861dd9fbf6745d186117a4 | [log] [tgz] |
---|---|---|
author | Antonio Barone <syntonyze@gmail.com> | Wed Apr 13 16:50:09 2022 +0200 |
committer | Antonio Barone <syntonyze@gmail.com> | Wed Apr 13 16:52:57 2022 +0200 |
tree | fc3193e04ea1d477a0a9dda5c88ec03668e79171 | |
parent | d960e2c9c3be33612e2cebf1f6103b2c3f6f2bf9 [diff] |
Remove duplicate stack trace logging logging at severe level with cause already shows the stack and the message of the error. Logging this explicitly would cause the stack trace to be logged twice. Change-Id: I3cc0a4fdc7349db3712a28d1e963c51cdd702193
diff --git a/src/main/java/com/gerritforge/gerrit/eventbroker/publisher/StreamEventPublisher.java b/src/main/java/com/gerritforge/gerrit/eventbroker/publisher/StreamEventPublisher.java index 5e49696..e1d1398 100644 --- a/src/main/java/com/gerritforge/gerrit/eventbroker/publisher/StreamEventPublisher.java +++ b/src/main/java/com/gerritforge/gerrit/eventbroker/publisher/StreamEventPublisher.java
@@ -84,8 +84,7 @@ brokerMetrics.get().incrementBrokerFailedToPublishMessage(); } catch (Throwable e) { log.atSevere().withCause(e).log( - "Failed to publish event '%s' to topic '%s' - error: %s - stack trace: %s", - event, streamEventTopic, e.getMessage(), e.getStackTrace()); + "Failed to publish event '%s' to topic '%s'", event, streamEventTopic); brokerMetrics.get().incrementBrokerFailedToPublishMessage(); } }