Clarify getLaggedReplicaMode()/laggedReplicaUsed() in LoadBalancer
authorAaron Schulz <aschulz@wikimedia.org>
Fri, 14 Apr 2017 01:59:39 +0000 (18:59 -0700)
committerAaron Schulz <aschulz@wikimedia.org>
Sat, 15 Apr 2017 06:50:36 +0000 (06:50 +0000)
Change-Id: I4ea9033d2d643c6758c49f34a69c9837598eea33

includes/libs/rdbms/loadbalancer/ILoadBalancer.php

index cbd0ff3..dbf653d 100644 (file)
@@ -444,13 +444,17 @@ interface ILoadBalancer {
        /**
         * @note This method will trigger a DB connection if not yet done
         * @param string|bool $domain Domain ID, or false for the current domain
-        * @return bool Whether the generic connection for reads is highly "lagged"
+        * @return bool Whether the database for generic connections this request is highly "lagged"
         */
        public function getLaggedReplicaMode( $domain = false );
 
        /**
+        * Checks whether the database for generic connections this request was both:
+        *   - a) Already choosen due to a prior connection attempt
+        *   - b) Considered highly "lagged"
+        *
         * @note This method will never cause a new DB connection
-        * @return bool Whether any generic connection used for reads was highly "lagged"
+        * @return bool
         */
        public function laggedReplicaUsed();