Merge "Fix various minor IDEA warnings in LoadBalancer"
authorjenkins-bot <jenkins-bot@gerrit.wikimedia.org>
Sat, 17 Dec 2016 06:41:34 +0000 (06:41 +0000)
committerGerrit Code Review <gerrit@wikimedia.org>
Sat, 17 Dec 2016 06:41:34 +0000 (06:41 +0000)
1  2 
includes/libs/rdbms/loadbalancer/LoadBalancer.php

@@@ -31,7 -31,7 +31,7 @@@ use Wikimedia\ScopedCallback
  class LoadBalancer implements ILoadBalancer {
        /** @var array[] Map of (server index => server config array) */
        private $mServers;
 -      /** @var IDatabase[][] Map of (local/foreignUsed/foreignFree => server index => IDatabase array) */
 +      /** @var IDatabase[][][] Map of local/foreignUsed/foreignFree => server index => IDatabase array */
        private $mConns;
        /** @var float[] Map of (server index => weight) */
        private $mLoads;
  
        /**
         * @param int $i
-        * @return IDatabase
+        * @return IDatabase|bool
         */
        public function getAnyOpenConnection( $i ) {
                foreach ( $this->mConns as $connsByServer ) {
                        if ( !empty( $connsByServer[$i] ) ) {
-                               return reset( $connsByServer[$i] );
+                               /** @var $serverConns IDatabase[] */
+                               $serverConns = $connsByServer[$i];
+                               return reset( $serverConns );
                        }
                }