Merge "rdbms: turn LoadBalancer waitTimeout default into a class constant"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Sun, 4 Feb 2018 18:22:20 +0000 (18:22 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Sun, 4 Feb 2018 18:22:21 +0000 (18:22 +0000)
includes/libs/rdbms/loadbalancer/LoadBalancer.php

index 1ba1a97..4ce7c6c 100644 (file)
@@ -122,6 +122,8 @@ class LoadBalancer implements ILoadBalancer {
 
        /** @var int Default 'maxLag' when unspecified */
        const MAX_LAG_DEFAULT = 10;
+       /** @var int Default 'waitTimeout' when unspecified */
+       const MAX_WAIT_DEFAULT = 10;
        /** @var int Seconds to cache master server read-only status */
        const TTL_CACHE_READONLY = 5;
 
@@ -151,7 +153,9 @@ class LoadBalancer implements ILoadBalancer {
                        : DatabaseDomain::newUnspecified();
                $this->setLocalDomain( $localDomain );
 
-               $this->mWaitTimeout = isset( $params['waitTimeout'] ) ? $params['waitTimeout'] : 10;
+               $this->mWaitTimeout = isset( $params['waitTimeout'] )
+                       ? $params['waitTimeout']
+                       : self::MAX_WAIT_DEFAULT;
 
                $this->mReadIndex = -1;
                $this->mConns = [