rdbms: turn LoadBalancer waitTimeout default into a class constant
authorAaron Schulz <aschulz@wikimedia.org>
Sat, 3 Feb 2018 03:30:48 +0000 (19:30 -0800)
committerAaron Schulz <aschulz@wikimedia.org>
Sat, 3 Feb 2018 03:30:48 +0000 (19:30 -0800)
Change-Id: Icd1e1aba1c45cb79a8249a9e16dce16f1c8183d2

includes/libs/rdbms/loadbalancer/LoadBalancer.php

index 864e6f0..a3361cd 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 = [