summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDavid Pursehouse <david.pursehouse@sonymobile.com>2016-02-03 17:12:03 +0900
committerDavid Pursehouse <david.pursehouse@sonymobile.com>2016-02-03 18:27:19 +0900
commit32e5af312857a541849ce9f690f5b68fbba81a70 (patch)
tree9ca52ae355082856f4f009315f01863365ef07c3
parente3c1af0768706f83f113a0832b0480ccd0072977 (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.java2
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;
}