rdbms: avoid duplicate spammy logging in LoadBalancer::getRandomNonLagged
authorAaron Schulz <aschulz@wikimedia.org>
Tue, 12 Feb 2019 18:52:31 +0000 (10:52 -0800)
committerAaron Schulz <aschulz@wikimedia.org>
Tue, 12 Feb 2019 18:52:31 +0000 (10:52 -0800)
This is already logged in LoadMonitor in getServerStates() in a less spammy way
(due to APC caching of server states).

Bug: T215611
Change-Id: Id70fdfa62eff9cb6446deea5e197f4c0af4928aa

includes/libs/rdbms/loadbalancer/LoadBalancer.php

index b3f9eff..f2e4e3d 100644 (file)
@@ -305,7 +305,7 @@ class LoadBalancer implements ILoadBalancer {
 
                                $host = $this->getServerName( $i );
                                if ( $lag === false && !is_infinite( $maxServerLag ) ) {
-                                       $this->replLogger->error(
+                                       $this->replLogger->debug(
                                                __METHOD__ .
                                                ": server {host} is not replicating?", [ 'host' => $host ] );
                                        unset( $loads[$i] );