Merge "New hook 'ParserTestGlobals'"
[lhc/web/wiklou.git] / includes / db / LoadBalancer.php
index bfbc9d8..c40ac0c 100644 (file)
@@ -1,6 +1,21 @@
 <?php
 /**
- * Database load balancing
+ * Database load balancing.
+ *
+ * This program is free software; you can redistribute it and/or modify
+ * it under the terms of the GNU General Public License as published by
+ * the Free Software Foundation; either version 2 of the License, or
+ * (at your option) any later version.
+ *
+ * This program is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
+ * GNU General Public License for more details.
+ *
+ * You should have received a copy of the GNU General Public License along
+ * with this program; if not, write to the Free Software Foundation, Inc.,
+ * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
+ * http://www.gnu.org/copyleft/gpl.html
  *
  * @file
  * @ingroup Database
@@ -50,8 +65,17 @@ class LoadBalancer {
                $this->mLaggedSlaveMode = false;
                $this->mErrorConnection = false;
                $this->mAllowLagged = false;
-               $this->mLoadMonitorClass = isset( $params['loadMonitor'] ) 
-                       ? $params['loadMonitor'] : 'LoadMonitor_MySQL';
+
+               if ( isset( $params['loadMonitor'] ) ) {
+                       $this->mLoadMonitorClass = $params['loadMonitor'];
+               } else {
+                       $master = reset( $params['servers'] );
+                       if ( isset( $master['type'] ) && $master['type'] === 'mysql' ) {
+                               $this->mLoadMonitorClass = 'LoadMonitor_MySQL';
+                       } else {
+                               $this->mLoadMonitorClass = 'LoadMonitor_Null';
+                       }
+               }
 
                foreach( $params['servers'] as $i => $server ) {
                        $this->mLoads[$i] = $server['load'];
@@ -81,6 +105,8 @@ class LoadBalancer {
 
        /**
         * Get or set arbitrary data used by the parent object, usually an LBFactory
+        * @param $x
+        * @return Mixed
         */
        function parentInfo( $x = null ) {
                return wfSetVar( $this->mParentInfo, $x );
@@ -90,7 +116,7 @@ class LoadBalancer {
         * Given an array of non-normalised probabilities, this function will select
         * an element and return the appropriate key
         *
-        * @param $weights
+        * @param $weights array
         *
         * @return int
         */
@@ -108,7 +134,7 @@ class LoadBalancer {
                        return false;
                }
                $max = mt_getrandmax();
-               $rand = mt_rand(0, $max) / $max * $sum;
+               $rand = mt_rand( 0, $max ) / $max * $sum;
 
                $sum = 0;
                foreach ( $weights as $i => $w ) {
@@ -121,7 +147,7 @@ class LoadBalancer {
        }
 
        /**
-        * @param $loads
+        * @param $loads array
         * @param $wiki bool
         * @return bool|int|string
         */
@@ -131,10 +157,10 @@ class LoadBalancer {
                foreach ( $lags as $i => $lag ) {
                        if ( $i != 0 ) {
                                if ( $lag === false ) {
-                                       wfDebug( "Server #$i is not replicating\n" );
+                                       wfDebugLog( 'replication', "Server #$i is not replicating\n" );
                                        unset( $loads[$i] );
                                } elseif ( isset( $this->mServers[$i]['max lag'] ) && $lag > $this->mServers[$i]['max lag'] ) {
-                                       wfDebug( "Server #$i is excessively lagged ($lag seconds)\n" );
+                                       wfDebugLog( 'replication', "Server #$i is excessively lagged ($lag seconds)\n" );
                                        unset( $loads[$i] );
                                }
                        }
@@ -232,7 +258,9 @@ class LoadBalancer {
                                        $i = $this->getRandomNonLagged( $currentLoads, $wiki );
                                        if ( $i === false && count( $currentLoads ) != 0 )  {
                                                # All slaves lagged. Switch to read-only mode
-                                               $wgReadOnly = wfMessage( 'readonly_lag' )->useDatabase( false )->plain();
+                                               wfDebugLog( 'replication', "All slaves lagged. Switch to read-only mode\n" );
+                                               $wgReadOnly = 'The database has been automatically locked ' .
+                                                       'while the slave database servers catch up to the master';
                                                $i = $this->pickRandom( $currentLoads );
                                                $laggedSlaveMode = true;
                                        }
@@ -240,7 +268,7 @@ class LoadBalancer {
 
                                if ( $i === false ) {
                                        # pickRandom() returned false
-                                       # This is permanent and means the configuration or the load monitor 
+                                       # This is permanent and means the configuration or the load monitor
                                        # wants us to return false.
                                        wfDebugLog( 'connect', __METHOD__.": pickRandom() returned false\n" );
                                        wfProfileOut( __METHOD__ );
@@ -258,7 +286,7 @@ class LoadBalancer {
                                }
 
                                // Perform post-connection backoff
-                               $threshold = isset( $this->mServers[$i]['max threads'] ) 
+                               $threshold = isset( $this->mServers[$i]['max threads'] )
                                        ? $this->mServers[$i]['max threads'] : false;
                                $backoff = $this->getLoadMonitor()->postConnectionBackoff( $conn, $threshold );
 
@@ -267,7 +295,7 @@ class LoadBalancer {
                                if ( $wiki !== false ) {
                                        $this->reuseConnection( $conn );
                                }
-                               
+
                                if ( $backoff ) {
                                        # Post-connection overload, don't use this server for now
                                        $totalThreadsConnected += $backoff;
@@ -323,6 +351,8 @@ class LoadBalancer {
 
        /**
         * Wait for a specified number of microseconds, and return the period waited
+        * @param $t int
+        * @return int
         */
        function sleep( $t ) {
                wfProfileIn( __METHOD__ );
@@ -336,6 +366,7 @@ class LoadBalancer {
         * Set the master wait position
         * If a DB_SLAVE connection has been opened already, waits
         * Otherwise sets a variable telling it to wait if such a connection is opened
+        * @param $pos int
         */
        public function waitFor( $pos ) {
                wfProfileIn( __METHOD__ );
@@ -350,9 +381,10 @@ class LoadBalancer {
                }
                wfProfileOut( __METHOD__ );
        }
-       
+
        /**
         * Set the master wait position and wait for ALL slaves to catch up to it
+        * @param $pos int
         */
        public function waitForAll( $pos ) {
                wfProfileIn( __METHOD__ );
@@ -367,7 +399,8 @@ class LoadBalancer {
         * Get any open connection to a given server index, local or foreign
         * Returns false if there is no connection open
         *
-        * @return DatabaseBase
+        * @param $i int
+        * @return DatabaseBase|bool False on failure
         */
        function getAnyOpenConnection( $i ) {
                foreach ( $this->mConns as $conns ) {
@@ -380,6 +413,9 @@ class LoadBalancer {
 
        /**
         * Wait for a given slave to catch up to the master pos stored in $this
+        * @param $index
+        * @param $open bool
+        * @return bool
         */
        function doWait( $index, $open = false ) {
                # Find a connection to wait on
@@ -413,11 +449,11 @@ class LoadBalancer {
        /**
         * Get a connection by index
         * This is the main entry point for this class.
-        * 
+        *
         * @param $i Integer: server index
         * @param $groups Array: query groups
         * @param $wiki String: wiki ID
-        * 
+        *
         * @return DatabaseBase
         */
        public function &getConnection( $i, $groups = array(), $wiki = false ) {
@@ -652,12 +688,14 @@ class LoadBalancer {
         * Returns a Database object whether or not the connection was successful.
         * @access private
         *
+        * @param $server
+        * @param $dbNameOverride bool
         * @return DatabaseBase
         */
        function reallyOpenConnection( $server, $dbNameOverride = false ) {
                if( !is_array( $server ) ) {
                        throw new MWException( 'You must update your load-balancing configuration. ' .
-                                              'See DefaultSettings.php entry for $wgDBservers.' );
+                               'See DefaultSettings.php entry for $wgDBservers.' );
                }
 
                $host = $server['host'];
@@ -669,7 +707,14 @@ class LoadBalancer {
 
                # Create object
                wfDebug( "Connecting to $host $dbname...\n" );
-               $db = DatabaseBase::newFromType( $server['type'], $server );
+               try {
+                       $db = DatabaseBase::factory( $server['type'], $server );
+               } catch ( DBConnectionError $e ) {
+                       // FIXME: This is probably the ugliest thing I have ever done to
+                       // PHP. I'm half-expecting it to segfault, just out of disgust. -- TS
+                       $db = $e->db;
+               }
+
                if ( $db->isOpen() ) {
                        wfDebug( "Connected to $host $dbname.\n" );
                } else {
@@ -685,12 +730,16 @@ class LoadBalancer {
                return $db;
        }
 
+       /**
+        * @param $conn
+        * @throws DBConnectionError
+        */
        function reportConnectionError( &$conn ) {
                wfProfileIn( __METHOD__ );
 
                if ( !is_object( $conn ) ) {
                        // No last connection, probably due to all servers being too busy
-                       wfLogDBError( "LB failure with no last connection\n" );
+                       wfLogDBError( "LB failure with no last connection. Connection error: {$this->mLastError}\n" );
                        $conn = new Database;
                        // If all servers were busy, mLastError will contain something sensible
                        throw new DBConnectionError( $conn, $this->mLastError );
@@ -702,6 +751,9 @@ class LoadBalancer {
                wfProfileOut( __METHOD__ );
        }
 
+       /**
+        * @return int
+        */
        function getWriterIndex() {
                return 0;
        }
@@ -709,6 +761,7 @@ class LoadBalancer {
        /**
         * Returns true if the specified index is a valid server index
         *
+        * @param $i
         * @return bool
         */
        function haveIndex( $i ) {
@@ -718,6 +771,7 @@ class LoadBalancer {
        /**
         * Returns true if the specified index is valid and has non-zero load
         *
+        * @param $i
         * @return bool
         */
        function isNonZeroLoad( $i ) {
@@ -736,6 +790,8 @@ class LoadBalancer {
        /**
         * Get the host name or IP address of the server with the specified index
         * Prefer a readable name if available.
+        * @param $i
+        * @return string
         */
        function getServerName( $i ) {
                if ( isset( $this->mServers[$i]['hostName'] ) ) {
@@ -749,6 +805,8 @@ class LoadBalancer {
 
        /**
         * Return the server info structure for a given index, or false if the index is invalid.
+        * @param $i
+        * @return bool
         */
        function getServerInfo( $i ) {
                if ( isset( $this->mServers[$i] ) ) {
@@ -760,6 +818,8 @@ class LoadBalancer {
 
        /**
         * Sets the server info structure for the given index. Entry at index $i is created if it doesn't exist
+        * @param $i
+        * @param $serverInfo
         */
        function setServerInfo( $i, $serverInfo ) {
                $this->mServers[$i] = $serverInfo;
@@ -810,8 +870,10 @@ class LoadBalancer {
         * Deprecated function, typo in function name
         *
         * @deprecated in 1.18
+        * @param $conn
         */
        function closeConnecton( $conn ) {
+               wfDeprecated( __METHOD__, '1.18' );
                $this->closeConnection( $conn );
        }
 
@@ -819,8 +881,7 @@ class LoadBalancer {
         * Close a connection
         * Using this function makes sure the LoadBalancer knows the connection is closed.
         * If you use $conn->close() directly, the load balancer won't update its state.
-        * @param $conn
-        * @return void
+        * @param $conn DatabaseBase
         */
        function closeConnection( $conn ) {
                $done = false;
@@ -848,7 +909,7 @@ class LoadBalancer {
                foreach ( $this->mConns as $conns2 ) {
                        foreach ( $conns2 as $conns3 ) {
                                foreach ( $conns3 as $conn ) {
-                                       $conn->commit();
+                                       $conn->commit( __METHOD__ );
                                }
                        }
                }
@@ -866,21 +927,32 @@ class LoadBalancer {
                        }
                        foreach ( $conns2[$masterIndex] as $conn ) {
                                if ( $conn->doneWrites() ) {
-                                       $conn->commit();
+                                       $conn->commit( __METHOD__ );
                                }
                        }
                }
        }
 
+       /**
+        * @param $value null
+        * @return Mixed
+        */
        function waitTimeout( $value = null ) {
                return wfSetVar( $this->mWaitTimeout, $value );
        }
 
+       /**
+        * @return bool
+        */
        function getLaggedSlaveMode() {
                return $this->mLaggedSlaveMode;
        }
 
-       /* Disables/enables lag checks */
+       /**
+        * Disables/enables lag checks
+        * @param $mode null
+        * @return bool
+        */
        function allowLagged( $mode = null ) {
                if ( $mode === null) {
                        return $this->mAllowLagged;
@@ -888,6 +960,9 @@ class LoadBalancer {
                $this->mAllowLagged = $mode;
        }
 
+       /**
+        * @return bool
+        */
        function pingAll() {
                $success = true;
                foreach ( $this->mConns as $conns2 ) {
@@ -904,6 +979,8 @@ class LoadBalancer {
 
        /**
         * Call a function with each open connection object
+        * @param $callback
+        * @param array $params
         */
        function forEachOpenConnection( $callback, $params = array() ) {
                foreach ( $this->mConns as $conns2 ) {
@@ -919,46 +996,87 @@ class LoadBalancer {
        /**
         * Get the hostname and lag time of the most-lagged slave.
         * This is useful for maintenance scripts that need to throttle their updates.
-        * May attempt to open connections to slaves on the default DB.
+        * May attempt to open connections to slaves on the default DB. If there is
+        * no lag, the maximum lag will be reported as -1.
+        *
         * @param $wiki string Wiki ID, or false for the default database
+        *
+        * @return array ( host, max lag, index of max lagged host )
         */
        function getMaxLag( $wiki = false ) {
                $maxLag = -1;
                $host = '';
-               foreach ( $this->mServers as $i => $conn ) {
-                       $conn = false;
-                       if ( $wiki === false ) {
-                               $conn = $this->getAnyOpenConnection( $i );
-                       }
-                       if ( !$conn ) {
-                               $conn = $this->openConnection( $i, $wiki );
-                       }
-                       if ( !$conn ) {
-                               continue;
-                       }
-                       $lag = $conn->getLag();
-                       if ( $lag > $maxLag ) {
-                               $maxLag = $lag;
-                               $host = $this->mServers[$i]['host'];
+               $maxIndex = 0;
+               if ( $this->getServerCount() > 1 ) { // no replication = no lag
+                       foreach ( $this->mServers as $i => $conn ) {
+                               $conn = false;
+                               if ( $wiki === false ) {
+                                       $conn = $this->getAnyOpenConnection( $i );
+                               }
+                               if ( !$conn ) {
+                                       $conn = $this->openConnection( $i, $wiki );
+                               }
+                               if ( !$conn ) {
+                                       continue;
+                               }
+                               $lag = $conn->getLag();
+                               if ( $lag > $maxLag ) {
+                                       $maxLag = $lag;
+                                       $host = $this->mServers[$i]['host'];
+                                       $maxIndex = $i;
+                               }
                        }
                }
-               return array( $host, $maxLag );
+               return array( $host, $maxLag, $maxIndex );
        }
 
        /**
         * Get lag time for each server
         * Results are cached for a short time in memcached, and indefinitely in the process cache
+        *
+        * @param $wiki
+        *
+        * @return array
         */
        function getLagTimes( $wiki = false ) {
                # Try process cache
                if ( isset( $this->mLagTimes ) ) {
                        return $this->mLagTimes;
                }
-               # No, send the request to the load monitor
-               $this->mLagTimes = $this->getLoadMonitor()->getLagTimes( array_keys( $this->mServers ), $wiki );
+               if ( $this->getServerCount() == 1 ) {
+                       # No replication
+                       $this->mLagTimes = array( 0 => 0 );
+               } else {
+                       # Send the request to the load monitor
+                       $this->mLagTimes = $this->getLoadMonitor()->getLagTimes(
+                               array_keys( $this->mServers ), $wiki );
+               }
                return $this->mLagTimes;
        }
 
+       /**
+        * Get the lag in seconds for a given connection, or zero if this load
+        * balancer does not have replication enabled.
+        *
+        * This should be used in preference to Database::getLag() in cases where
+        * replication may not be in use, since there is no way to determine if
+        * replication is in use at the connection level without running
+        * potentially restricted queries such as SHOW SLAVE STATUS. Using this
+        * function instead of Database::getLag() avoids a fatal error in this
+        * case on many installations.
+        *
+        * @param $conn DatabaseBase
+        *
+        * @return int
+        */
+       function safeGetLag( $conn ) {
+               if ( $this->getServerCount() == 1 ) {
+                       return 0;
+               } else {
+                       return $conn->getLag();
+               }
+       }
+
        /**
         * Clear the cache for getLagTimes
         */