From: Aaron Schulz <aschulz@wikimedia.org>
Date: Tue, 10 Jan 2017 21:20:02 +0000 (-0800)
Subject: Reduce WAN object cache logging spam for reject set() calls
X-Git-Tag: 1.31.0-rc.0~4355^2
X-Git-Url: http://git.cyclocoop.org/%22.htmlspecialchars%28%24url_syndic%29.%22?a=commitdiff_plain;h=8972104ba234a69232aacf8c829bc853d9f501da;p=lhc%2Fweb%2Fwiklou.git

Reduce WAN object cache logging spam for reject set() calls

Change-Id: I276c91b3b3b47d337f9c1d91a8348ca8f3d57194
---

diff --git a/includes/libs/objectcache/WANObjectCache.php b/includes/libs/objectcache/WANObjectCache.php
index 01a914912f..0db493de35 100644
--- a/includes/libs/objectcache/WANObjectCache.php
+++ b/includes/libs/objectcache/WANObjectCache.php
@@ -448,7 +448,7 @@ class WANObjectCache implements IExpiringStore, LoggerAwareInterface {
 				$wrapExtra[self::FLD_FLAGS] = self::FLG_STALE; // mark as stale
 			// Case B: any long-running transaction; ignore this set()
 			} elseif ( $age > self::MAX_READ_LAG ) {
-				$this->logger->warning( "Rejected set() for $key due to snapshot lag." );
+				$this->logger->info( "Rejected set() for $key due to snapshot lag." );
 
 				return true; // no-op the write for being unsafe
 			// Case C: high replication lag; lower TTL instead of ignoring all set()s
@@ -457,7 +457,7 @@ class WANObjectCache implements IExpiringStore, LoggerAwareInterface {
 				$this->logger->warning( "Lowered set() TTL for $key due to replication lag." );
 			// Case D: medium length request with medium replication lag; ignore this set()
 			} else {
-				$this->logger->warning( "Rejected set() for $key due to high read lag." );
+				$this->logger->info( "Rejected set() for $key due to high read lag." );
 
 				return true; // no-op the write for being unsafe
 			}