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>
This commit is contained in:
Edwin Kempin
2018-05-07 15:21:33 +02:00
parent 1ea84c5d83
commit 3db67d6050
1 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

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

@@ -129,7 +129,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());
@@ -140,7 +140,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());
}
}
}