diff options
author | David Pursehouse <david.pursehouse@sonymobile.com> | 2016-02-03 17:12:03 +0900 |
---|---|---|
committer | David Pursehouse <david.pursehouse@sonymobile.com> | 2016-02-03 18:27:19 +0900 |
commit | 32e5af312857a541849ce9f690f5b68fbba81a70 (patch) | |
tree | 9ca52ae355082856f4f009315f01863365ef07c3 | |
parent | e3c1af0768706f83f113a0832b0480ccd0072977 (diff) |
OutgoingEmail: Kill "Skipping delivery of email" logspam
The logs:
INFO: "Skipping delivery of email with no recipients" and
WARN: "Skipping delivery of email with no body"
don't add any value, and just fill up the logs. Remove them.
Change-Id: I66d19f8bd970bb9403849c4fa7bfb7f7f597b46d
-rw-r--r-- | gerrit-server/src/main/java/com/google/gerrit/server/mail/OutgoingEmail.java | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/gerrit-server/src/main/java/com/google/gerrit/server/mail/OutgoingEmail.java b/gerrit-server/src/main/java/com/google/gerrit/server/mail/OutgoingEmail.java index 970a36c687..9ca83a8912 100644 --- a/gerrit-server/src/main/java/com/google/gerrit/server/mail/OutgoingEmail.java +++ b/gerrit-server/src/main/java/com/google/gerrit/server/mail/OutgoingEmail.java @@ -290,7 +290,6 @@ public abstract class OutgoingEmail { protected boolean shouldSendMessage() { if (body.length() == 0) { // If we have no message body, don't send. - log.warn("Skipping delivery of email with no body"); return false; } @@ -298,7 +297,6 @@ public abstract class OutgoingEmail { // If we have nobody to send this message to, then all of our // selection filters previously for this type of message were // unable to match a destination. Don't bother sending it. - log.info("Skipping delivery of email with no recipients"); return false; } |