opendev/gerrit
Code Issues Proposed changes

EventUtil: Always say 'event listener' in logs

Browse Source 
The debug logs say 'event listener', but the warning logs just
'listener'. Make this consistent and always use 'event listener' which
is more precise.

Change-Id: I447a00cce0b11e5337801ce5d84c7954c1340f59
Signed-off-by: Edwin Kempin <ekempin@google.com>
(cherry picked from commit 3db67d6050)
This commit is contained in:
Edwin Kempin
2018-05-07 15:21:33 +02:00
committed by David Pursehouse
parent e149c14239
commit 1ce1cb17d9
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
gerrit-server/src/main/java/com/google/gerrit/server/extensions/events/EventUtil.java
View File

@@ -124,7 +124,7 @@ public class EventUtil {
error);
} else {
log.warn(
"Error in listener {} for event {}: {}",
"Error in event listener {} for event {}: {}",
listener.getClass().getName(),
event.getClass().getName(),
error.getMessage());
@@ -135,7 +135,7 @@ public class EventUtil {
if (log.isDebugEnabled()) {
log.debug(String.format("Error in event listener %s", listener.getClass().getName()), error);
} else {
log.warn("Error in listener {}: {}", listener.getClass().getName(), error.getMessage());
log.warn("Error in event listener {}: {}", listener.getClass().getName(), error.getMessage());
}
}
}