LBFactory::commitMasterChanges() - Code Metrics - Inspection of "Daily Inspection: Merge "Convert Special:WithoutIn..." - wikimedia/mediawiki - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Branch — master (86dc85)

unknown

created 2016-04-19 17:14 UTC

LBFactory::commitMasterChanges() A

↳ Parent: LBFactory

Complexity

Conditions	4
Paths	2

Size

Total Lines	18
Code Lines	11

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
dl	0
loc	18
rs	9.2
c	1
b	0
f	0
cc	4
eloc	11
nc	2
nop	2

<?php
/**
 * Generator of database load balancing objects.
 *
 * 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
 */

use Psr\Log\LoggerInterface;
use MediaWiki\Logger\LoggerFactory;

/**
 * An interface for generating database load balancers
 * @ingroup Database
 */
abstract class LBFactory {
	/** @var ChronologyProtector */
	protected $chronProt;

	/** @var TransactionProfiler */
	protected $trxProfiler;

	/** @var LoggerInterface */
	protected $logger;

	/** @var LBFactory */
	private static $instance;

	/** @var string|bool Reason all LBs are read-only or false if not */
	protected $readOnlyReason = false;

	const SHUTDOWN_NO_CHRONPROT = 1; // don't save ChronologyProtector positions (for async code)

	/**
	 * Construct a factory based on a configuration array (typically from $wgLBFactoryConf)
	 * @param array $conf
	 */
	public function __construct( array $conf ) {
		if ( isset( $conf['readOnlyReason'] ) && is_string( $conf['readOnlyReason'] ) ) {
			$this->readOnlyReason = $conf['readOnlyReason'];
		}

		$this->chronProt = $this->newChronologyProtector();
		$this->trxProfiler = Profiler::instance()->getTransactionProfiler();
		$this->logger = LoggerFactory::getInstance( 'DBTransaction' );
	}

	/**
	 * Disables all access to the load balancer, will cause all database access
	 * to throw a DBAccessError
	 */
	public static function disableBackend() {
		global $wgLBFactoryConf;
		self::$instance = new LBFactoryFake( $wgLBFactoryConf );
	}

	/**
	 * Get an LBFactory instance
	 *
	 * @return LBFactory
	 */
	public static function singleton() {
		global $wgLBFactoryConf;

		if ( is_null( self::$instance ) ) {
			$class = self::getLBFactoryClass( $wgLBFactoryConf );
			$config = $wgLBFactoryConf;
			if ( !isset( $config['readOnlyReason'] ) ) {
				$config['readOnlyReason'] = wfConfiguredReadOnlyReason();
			}
			self::$instance = new $class( $config );
		}

		return self::$instance;
	}

	/**
	 * Returns the LBFactory class to use and the load balancer configuration.
	 *
	 * @param array $config (e.g. $wgLBFactoryConf)
	 * @return string Class name
	 */
	public static function getLBFactoryClass( array $config ) {
		// For configuration backward compatibility after removing
		// underscores from class names in MediaWiki 1.23.
		$bcClasses = [
			'LBFactory_Simple' => 'LBFactorySimple',
			'LBFactory_Single' => 'LBFactorySingle',
			'LBFactory_Multi' => 'LBFactoryMulti',
			'LBFactory_Fake' => 'LBFactoryFake',
		];

		$class = $config['class'];

		if ( isset( $bcClasses[$class] ) ) {
			$class = $bcClasses[$class];
			wfDeprecated(
				'$wgLBFactoryConf must be updated. See RELEASE-NOTES for details',
				'1.23'
			);
		}

		return $class;
	}

	/**
	 * Shut down, close connections and destroy the cached instance.
	 */
	public static function destroyInstance() {
		if ( self::$instance ) {
			self::$instance->shutdown();
			self::$instance->forEachLBCallMethod( 'closeAll' );
			self::$instance = null;
		}
	}

	/**
	 * Set the instance to be the given object
	 *
	 * @param LBFactory $instance
	 */
	public static function setInstance( $instance ) {
		self::destroyInstance();
		self::$instance = $instance;
	}

	/**
	 * Create a new load balancer object. The resulting object will be untracked,
	 * not chronology-protected, and the caller is responsible for cleaning it up.
	 *
	 * @param bool|string $wiki Wiki ID, or false for the current wiki
	 * @return LoadBalancer
	 */
	abstract public function newMainLB( $wiki = false );

	/**
	 * Get a cached (tracked) load balancer object.
	 *
	 * @param bool|string $wiki Wiki ID, or false for the current wiki
	 * @return LoadBalancer
	 */
	abstract public function getMainLB( $wiki = false );

	/**
	 * Create a new load balancer for external storage. The resulting object will be
	 * untracked, not chronology-protected, and the caller is responsible for
	 * cleaning it up.
	 *
	 * @param string $cluster External storage cluster, or false for core
	 * @param bool|string $wiki Wiki ID, or false for the current wiki
	 * @return LoadBalancer
	 */
	abstract protected function newExternalLB( $cluster, $wiki = false );

	/**
	 * Get a cached (tracked) load balancer for external storage
	 *
	 * @param string $cluster External storage cluster, or false for core
	 * @param bool|string $wiki Wiki ID, or false for the current wiki
	 * @return LoadBalancer
	 */
	abstract public function &getExternalLB( $cluster, $wiki = false );

	/**
	 * Execute a function for each tracked load balancer
	 * The callback is called with the load balancer as the first parameter,
	 * and $params passed as the subsequent parameters.
	 *
	 * @param callable $callback
	 * @param array $params
	 */
	abstract public function forEachLB( $callback, array $params = [] );

	/**
	 * Prepare all tracked load balancers for shutdown
	 * @param integer $flags Supports SHUTDOWN_* flags
	 * STUB
	 */
	public function shutdown( $flags = 0 ) {
	}

	/**
	 * Call a method of each tracked load balancer
	 *
	 * @param string $methodName
	 * @param array $args
	 */
	private function forEachLBCallMethod( $methodName, array $args = [] ) {
		$this->forEachLB(
			function ( LoadBalancer $loadBalancer, $methodName, array $args ) {
				call_user_func_array( [ $loadBalancer, $methodName ], $args );
			},
			[ $methodName, $args ]
		);
	}

	/**
	 * Commit on all connections. Done for two reasons:
	 * 1. To commit changes to the masters.
	 * 2. To release the snapshot on all connections, master and slave.
	 * @param string $fname Caller name
	 */
	public function commitAll( $fname = __METHOD__ ) {
		$this->logMultiDbTransaction();

		$start = microtime( true );
		$this->forEachLBCallMethod( 'commitAll', [ $fname ] );
		$timeMs = 1000 * ( microtime( true ) - $start );

		RequestContext::getMain()->getStats()->timing( "db.commit-all", $timeMs );
	}

	/**
	 * Commit changes on all master connections
	 * @param string $fname Caller name
	 * @param array $options Options map:
	 *   - maxWriteDuration: abort if more than this much time was spent in write queries
	 */
	public function commitMasterChanges( $fname = __METHOD__, array $options = [] ) {
		$limit = isset( $options['maxWriteDuration'] ) ? $options['maxWriteDuration'] : 0;

		$this->logMultiDbTransaction();
		$this->forEachLB( function ( LoadBalancer $lb ) use ( $limit ) {
			$lb->forEachOpenConnection( function ( IDatabase $db ) use ( $limit ) {
				$time = $db->pendingWriteQueryDuration();
				if ( $limit > 0 && $time > $limit ) {
					throw new DBTransactionError(
						$db,
						wfMessage( 'transaction-duration-limit-exceeded', $time, $limit )->text()
					);
				}
			} );
		} );

		$this->forEachLBCallMethod( 'commitMasterChanges', [ $fname ] );
	}

	/**
	 * Rollback changes on all master connections
	 * @param string $fname Caller name
	 * @since 1.23
	 */
	public function rollbackMasterChanges( $fname = __METHOD__ ) {
		$this->forEachLBCallMethod( 'rollbackMasterChanges', [ $fname ] );
	}

	/**
	 * Log query info if multi DB transactions are going to be committed now
	 */
	private function logMultiDbTransaction() {
		$callersByDB = [];
		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$callersByDB ) {
			$masterName = $lb->getServerName( $lb->getWriterIndex() );
			$callers = $lb->pendingMasterChangeCallers();
			if ( $callers ) {

				$callersByDB[$masterName] = $callers;
			}
		} );

		if ( count( $callersByDB ) >= 2 ) {
			$dbs = implode( ', ', array_keys( $callersByDB ) );
			$msg = "Multi-DB transaction [{$dbs}]:\n";
			foreach ( $callersByDB as $db => $callers ) {
				$msg .= "$db: " . implode( '; ', $callers ) . "\n";
			}
			$this->logger->info( $msg );
		}
	}

	/**
	 * Determine if any master connection has pending changes
	 * @return bool
	 * @since 1.23
	 */
	public function hasMasterChanges() {
		$ret = false;
		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$ret ) {
			$ret = $ret || $lb->hasMasterChanges();
		} );

		return $ret;
	}

	/**
	 * Detemine if any lagged slave connection was used
	 * @since 1.27
	 * @return bool
	 */
	public function laggedSlaveUsed() {
		$ret = false;
		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$ret ) {
			$ret = $ret || $lb->laggedSlaveUsed();
		} );

		return $ret;
	}

	/**
	 * Determine if any master connection has pending/written changes from this request
	 * @return bool
	 * @since 1.27
	 */
	public function hasOrMadeRecentMasterChanges() {
		$ret = false;
		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$ret ) {
			$ret = $ret || $lb->hasOrMadeRecentMasterChanges();
		} );
		return $ret;
	}

	/**
	 * Waits for the slave DBs to catch up to the current master position
	 *
	 * Use this when updating very large numbers of rows, as in maintenance scripts,
	 * to avoid causing too much lag. Of course, this is a no-op if there are no slaves.
	 *
	 * By default this waits on all DB clusters actually used in this request.
	 * This makes sense when lag being waiting on is caused by the code that does this check.
	 * In that case, setting "ifWritesSince" can avoid the overhead of waiting for clusters
	 * that were not changed since the last wait check. To forcefully wait on a specific cluster
	 * for a given wiki, use the 'wiki' parameter. To forcefully wait on an "external" cluster,
	 * use the "cluster" parameter.
	 *
	 * Never call this function after a large DB write that is *still* in a transaction.
	 * It only makes sense to call this after the possible lag inducing changes were committed.
	 *
	 * @param array $opts Optional fields that include:
	 *   - wiki : wait on the load balancer DBs that handles the given wiki
	 *   - cluster : wait on the given external load balancer DBs
	 *   - timeout : Max wait time. Default: ~60 seconds
	 *   - ifWritesSince: Only wait if writes were done since this UNIX timestamp
	 * @throws DBReplicationWaitError If a timeout or error occured waiting on a DB cluster
	 * @since 1.27
	 */
	public function waitForReplication( array $opts = [] ) {
		$opts += [
			'wiki' => false,
			'cluster' => false,
			'timeout' => 60,
			'ifWritesSince' => null
		];

		// Figure out which clusters need to be checked
		/** @var LoadBalancer[] $lbs */
		$lbs = [];
		if ( $opts['cluster'] !== false ) {
			$lbs[] = $this->getExternalLB( $opts['cluster'] );
		} elseif ( $opts['wiki'] !== false ) {
			$lbs[] = $this->getMainLB( $opts['wiki'] );
		} else {
			$this->forEachLB( function ( LoadBalancer $lb ) use ( &$lbs ) {
				$lbs[] = $lb;
			} );
			if ( !$lbs ) {

				return; // nothing actually used
			}
		}

		// Get all the master positions of applicable DBs right now.
		// This can be faster since waiting on one cluster reduces the
		// time needed to wait on the next clusters.
		$masterPositions = array_fill( 0, count( $lbs ), false );
		foreach ( $lbs as $i => $lb ) {
			if ( $lb->getServerCount() <= 1 ) {
				// Bug 27975 - Don't try to wait for slaves if there are none
				// Prevents permission error when getting master position
				continue;
			} elseif ( $opts['ifWritesSince']
				&& $lb->lastMasterChangeTimestamp() < $opts['ifWritesSince']
			) {
				continue; // no writes since the last wait
			}
			$masterPositions[$i] = $lb->getMasterPos();
		}

		$failed = [];
		foreach ( $lbs as $i => $lb ) {
			if ( $masterPositions[$i] ) {
				// The DBMS may not support getMasterPos() or the whole
				// load balancer might be fake (e.g. $wgAllDBsAreLocalhost).
				if ( !$lb->waitForAll( $masterPositions[$i], $opts['timeout'] ) ) {
					$failed[] = $lb->getServerName( $lb->getWriterIndex() );
				}
			}
		}

		if ( $failed ) {

			throw new DBReplicationWaitError(
				"Could not wait for slaves to catch up to " .
				implode( ', ', $failed )
			);
		}
	}

	/**
	 * Disable the ChronologyProtector for all load balancers
	 *
	 * This can be called at the start of special API entry points
	 *
	 * @since 1.27
	 */
	public function disableChronologyProtection() {
		$this->chronProt->setEnabled( false );
	}

	/**
	 * @return ChronologyProtector
	 */
	protected function newChronologyProtector() {
		$request = RequestContext::getMain()->getRequest();
		$chronProt = new ChronologyProtector(
			ObjectCache::getMainStashInstance(),
			[
				'ip' => $request->getIP(),
				'agent' => $request->getHeader( 'User-Agent' )
			]
		);
		if ( PHP_SAPI === 'cli' ) {
			$chronProt->setEnabled( false );
		} elseif ( $request->getHeader( 'ChronologyProtection' ) === 'false' ) {
			// Request opted out of using position wait logic. This is useful for requests
			// done by the job queue or background ETL that do not have a meaningful session.
			$chronProt->setWaitEnabled( false );
		}

		return $chronProt;
	}

	/**
	 * @param ChronologyProtector $cp
	 */
	protected function shutdownChronologyProtector( ChronologyProtector $cp ) {
		// Get all the master positions needed
		$this->forEachLB( function ( LoadBalancer $lb ) use ( $cp ) {
			$cp->shutdownLB( $lb );
		} );
		// Write them to the stash
		$unsavedPositions = $cp->shutdown();
		// If the positions failed to write to the stash, at least wait on local datacenter
		// slaves to catch up before responding. Even if there are several DCs, this increases
		// the chance that the user will see their own changes immediately afterwards. As long
		// as the sticky DC cookie applies (same domain), this is not even an issue.
		$this->forEachLB( function ( LoadBalancer $lb ) use ( $unsavedPositions ) {
			$masterName = $lb->getServerName( $lb->getWriterIndex() );
			if ( isset( $unsavedPositions[$masterName] ) ) {
				$lb->waitForAll( $unsavedPositions[$masterName] );
			}
		} );
	}
}

/**
 * Exception class for attempted DB access
 */
class DBAccessError extends MWException {
	public function __construct() {
		parent::__construct( "Mediawiki tried to access the database via wfGetDB(). " .
			"This is not allowed." );
	}
}

/**
 * Exception class for replica DB wait timeouts
 */
class DBReplicationWaitError extends Exception {
}


1		<?php
2		/**
3		* Generator of database load balancing objects.
4		*
5		* This program is free software; you can redistribute it and/or modify
6		* it under the terms of the GNU General Public License as published by
7		* the Free Software Foundation; either version 2 of the License, or
8		* (at your option) any later version.
9		*
10		* This program is distributed in the hope that it will be useful,
11		* but WITHOUT ANY WARRANTY; without even the implied warranty of
12		* MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
13		* GNU General Public License for more details.
14		*
15		* You should have received a copy of the GNU General Public License along
16		* with this program; if not, write to the Free Software Foundation, Inc.,
17		* 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301, USA.
18		* http://www.gnu.org/copyleft/gpl.html
19		*
20		* @file
21		* @ingroup Database
22		*/
23
24		use Psr\Log\LoggerInterface;
25		use MediaWiki\Logger\LoggerFactory;
26
27		/**
28		* An interface for generating database load balancers
29		* @ingroup Database
30		*/
31		abstract class LBFactory {
32		/** @var ChronologyProtector */
33		protected $chronProt;
34
35		/** @var TransactionProfiler */
36		protected $trxProfiler;
37
38		/** @var LoggerInterface */
39		protected $logger;
40
41		/** @var LBFactory */
42		private static $instance;
43
44		/** @var string\|bool Reason all LBs are read-only or false if not */
45		protected $readOnlyReason = false;
46
47		const SHUTDOWN_NO_CHRONPROT = 1; // don't save ChronologyProtector positions (for async code)
48
49		/**
50		* Construct a factory based on a configuration array (typically from $wgLBFactoryConf)
51		* @param array $conf
52		*/
53		public function __construct( array $conf ) {
54	View Code Duplication	if ( isset( $conf['readOnlyReason'] ) && is_string( $conf['readOnlyReason'] ) ) {
55		$this->readOnlyReason = $conf['readOnlyReason'];
56		}
57
58		$this->chronProt = $this->newChronologyProtector();
59		$this->trxProfiler = Profiler::instance()->getTransactionProfiler();
60		$this->logger = LoggerFactory::getInstance( 'DBTransaction' );
61		}
62
63		/**
64		* Disables all access to the load balancer, will cause all database access
65		* to throw a DBAccessError
66		*/
67		public static function disableBackend() {
68		global $wgLBFactoryConf;
69		self::$instance = new LBFactoryFake( $wgLBFactoryConf );
70		}
71
72		/**
73		* Get an LBFactory instance
74		*
75		* @return LBFactory
76		*/
77		public static function singleton() {
78		global $wgLBFactoryConf;
79
80		if ( is_null( self::$instance ) ) {
81		$class = self::getLBFactoryClass( $wgLBFactoryConf );
82		$config = $wgLBFactoryConf;
83		if ( !isset( $config['readOnlyReason'] ) ) {
84		$config['readOnlyReason'] = wfConfiguredReadOnlyReason();
85		}
86		self::$instance = new $class( $config );
87		}
88
89		return self::$instance;
90		}
91
92		/**
93		* Returns the LBFactory class to use and the load balancer configuration.
94		*
95		* @param array $config (e.g. $wgLBFactoryConf)
96		* @return string Class name
97		*/
98		public static function getLBFactoryClass( array $config ) {
99		// For configuration backward compatibility after removing
100		// underscores from class names in MediaWiki 1.23.
101		$bcClasses = [
102		'LBFactory_Simple' => 'LBFactorySimple',
103		'LBFactory_Single' => 'LBFactorySingle',
104		'LBFactory_Multi' => 'LBFactoryMulti',
105		'LBFactory_Fake' => 'LBFactoryFake',
106		];
107
108		$class = $config['class'];
109
110		if ( isset( $bcClasses[$class] ) ) {
111		$class = $bcClasses[$class];
112		wfDeprecated(
113		'$wgLBFactoryConf must be updated. See RELEASE-NOTES for details',
114		'1.23'
115		);
116		}
117
118		return $class;
119		}
120
121		/**
122		* Shut down, close connections and destroy the cached instance.
123		*/
124		public static function destroyInstance() {
125		if ( self::$instance ) {
126		self::$instance->shutdown();
127		self::$instance->forEachLBCallMethod( 'closeAll' );
128		self::$instance = null;
129		}
130		}
131
132		/**
133		* Set the instance to be the given object
134		*
135		* @param LBFactory $instance
136		*/
137		public static function setInstance( $instance ) {
138		self::destroyInstance();
139		self::$instance = $instance;
140		}
141
142		/**
143		* Create a new load balancer object. The resulting object will be untracked,
144		* not chronology-protected, and the caller is responsible for cleaning it up.
145		*
146		* @param bool\|string $wiki Wiki ID, or false for the current wiki
147		* @return LoadBalancer
148		*/
149		abstract public function newMainLB( $wiki = false );
150
151		/**
152		* Get a cached (tracked) load balancer object.
153		*
154		* @param bool\|string $wiki Wiki ID, or false for the current wiki
155		* @return LoadBalancer
156		*/
157		abstract public function getMainLB( $wiki = false );
158
159		/**
160		* Create a new load balancer for external storage. The resulting object will be
161		* untracked, not chronology-protected, and the caller is responsible for
162		* cleaning it up.
163		*
164		* @param string $cluster External storage cluster, or false for core
165		* @param bool\|string $wiki Wiki ID, or false for the current wiki
166		* @return LoadBalancer
167		*/
168		abstract protected function newExternalLB( $cluster, $wiki = false );
169
170		/**
171		* Get a cached (tracked) load balancer for external storage
172		*
173		* @param string $cluster External storage cluster, or false for core
174		* @param bool\|string $wiki Wiki ID, or false for the current wiki
175		* @return LoadBalancer
176		*/
177		abstract public function &getExternalLB( $cluster, $wiki = false );
178
179		/**
180		* Execute a function for each tracked load balancer
181		* The callback is called with the load balancer as the first parameter,
182		* and $params passed as the subsequent parameters.
183		*
184		* @param callable $callback
185		* @param array $params
186		*/
187		abstract public function forEachLB( $callback, array $params = [] );
188
189		/**
190		* Prepare all tracked load balancers for shutdown
191		* @param integer $flags Supports SHUTDOWN_* flags
192		* STUB
193		*/
194		public function shutdown( $flags = 0 ) {
195		}
196
197		/**
198		* Call a method of each tracked load balancer
199		*
200		* @param string $methodName
201		* @param array $args
202		*/
203		private function forEachLBCallMethod( $methodName, array $args = [] ) {
204		$this->forEachLB(
205		function ( LoadBalancer $loadBalancer, $methodName, array $args ) {
206		call_user_func_array( [ $loadBalancer, $methodName ], $args );
207		},
208		[ $methodName, $args ]
209		);
210		}
211
212		/**
213		* Commit on all connections. Done for two reasons:
214		* 1. To commit changes to the masters.
215		* 2. To release the snapshot on all connections, master and slave.
216		* @param string $fname Caller name
217		*/
218		public function commitAll( $fname = __METHOD__ ) {
219		$this->logMultiDbTransaction();
220
221		$start = microtime( true );
222		$this->forEachLBCallMethod( 'commitAll', [ $fname ] );
223		$timeMs = 1000 * ( microtime( true ) - $start );
224
225		RequestContext::getMain()->getStats()->timing( "db.commit-all", $timeMs );
226		}
227
228		/**
229		* Commit changes on all master connections
230		* @param string $fname Caller name
231		* @param array $options Options map:
232		* - maxWriteDuration: abort if more than this much time was spent in write queries
233		*/
234		public function commitMasterChanges( $fname = __METHOD__, array $options = [] ) {
235		$limit = isset( $options['maxWriteDuration'] ) ? $options['maxWriteDuration'] : 0;
236
237		$this->logMultiDbTransaction();
238		$this->forEachLB( function ( LoadBalancer $lb ) use ( $limit ) {
239		$lb->forEachOpenConnection( function ( IDatabase $db ) use ( $limit ) {
240		$time = $db->pendingWriteQueryDuration();
241		if ( $limit > 0 && $time > $limit ) {
242		throw new DBTransactionError(
243		$db,
244		wfMessage( 'transaction-duration-limit-exceeded', $time, $limit )->text()
245		);
246		}
247		} );
248		} );
249
250		$this->forEachLBCallMethod( 'commitMasterChanges', [ $fname ] );
251		}
252
253		/**
254		* Rollback changes on all master connections
255		* @param string $fname Caller name
256		* @since 1.23
257		*/
258		public function rollbackMasterChanges( $fname = __METHOD__ ) {
259		$this->forEachLBCallMethod( 'rollbackMasterChanges', [ $fname ] );
260		}
261
262		/**
263		* Log query info if multi DB transactions are going to be committed now
264		*/
265		private function logMultiDbTransaction() {
266		$callersByDB = [];
267		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$callersByDB ) {
268		$masterName = $lb->getServerName( $lb->getWriterIndex() );
269		$callers = $lb->pendingMasterChangeCallers();
270		if ( $callers ) {
		0 ignored issues – show Bug Best Practice introduced 2016-01-25 18:14 UTC by Report Bug Copy Issue Report The expression `$callers` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
271		$callersByDB[$masterName] = $callers;
272		}
273		} );
274
275		if ( count( $callersByDB ) >= 2 ) {
276		$dbs = implode( ', ', array_keys( $callersByDB ) );
277		$msg = "Multi-DB transaction [{$dbs}]:\n";
278		foreach ( $callersByDB as $db => $callers ) {
279		$msg .= "$db: " . implode( '; ', $callers ) . "\n";
280		}
281		$this->logger->info( $msg );
282		}
283		}
284
285		/**
286		* Determine if any master connection has pending changes
287		* @return bool
288		* @since 1.23
289		*/
290		public function hasMasterChanges() {
291		$ret = false;
292		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$ret ) {
293		$ret = $ret \|\| $lb->hasMasterChanges();
294		} );
295
296		return $ret;
297		}
298
299		/**
300		* Detemine if any lagged slave connection was used
301		* @since 1.27
302		* @return bool
303		*/
304		public function laggedSlaveUsed() {
305		$ret = false;
306		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$ret ) {
307		$ret = $ret \|\| $lb->laggedSlaveUsed();
308		} );
309
310		return $ret;
311		}
312
313		/**
314		* Determine if any master connection has pending/written changes from this request
315		* @return bool
316		* @since 1.27
317		*/
318		public function hasOrMadeRecentMasterChanges() {
319		$ret = false;
320		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$ret ) {
321		$ret = $ret \|\| $lb->hasOrMadeRecentMasterChanges();
322		} );
323		return $ret;
324		}
325
326		/**
327		* Waits for the slave DBs to catch up to the current master position
328		*
329		* Use this when updating very large numbers of rows, as in maintenance scripts,
330		* to avoid causing too much lag. Of course, this is a no-op if there are no slaves.
331		*
332		* By default this waits on all DB clusters actually used in this request.
333		* This makes sense when lag being waiting on is caused by the code that does this check.
334		* In that case, setting "ifWritesSince" can avoid the overhead of waiting for clusters
335		* that were not changed since the last wait check. To forcefully wait on a specific cluster
336		* for a given wiki, use the 'wiki' parameter. To forcefully wait on an "external" cluster,
337		* use the "cluster" parameter.
338		*
339		* Never call this function after a large DB write that is still in a transaction.
340		* It only makes sense to call this after the possible lag inducing changes were committed.
341		*
342		* @param array $opts Optional fields that include:
343		* - wiki : wait on the load balancer DBs that handles the given wiki
344		* - cluster : wait on the given external load balancer DBs
345		* - timeout : Max wait time. Default: ~60 seconds
346		* - ifWritesSince: Only wait if writes were done since this UNIX timestamp
347		* @throws DBReplicationWaitError If a timeout or error occured waiting on a DB cluster
348		* @since 1.27
349		*/
350		public function waitForReplication( array $opts = [] ) {
351		$opts += [
352		'wiki' => false,
353		'cluster' => false,
354		'timeout' => 60,
355		'ifWritesSince' => null
356		];
357
358		// Figure out which clusters need to be checked
359		/** @var LoadBalancer[] $lbs */
360		$lbs = [];
361		if ( $opts['cluster'] !== false ) {
362		$lbs[] = $this->getExternalLB( $opts['cluster'] );
363		} elseif ( $opts['wiki'] !== false ) {
364		$lbs[] = $this->getMainLB( $opts['wiki'] );
365		} else {
366		$this->forEachLB( function ( LoadBalancer $lb ) use ( &$lbs ) {
367		$lbs[] = $lb;
368		} );
369		if ( !$lbs ) {
		0 ignored issues – show Bug Best Practice introduced 2016-01-25 18:14 UTC by Report Bug Copy Issue Report The expression `$lbs` of type `LoadBalancer[]` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
370		return; // nothing actually used
371		}
372		}
373
374		// Get all the master positions of applicable DBs right now.
375		// This can be faster since waiting on one cluster reduces the
376		// time needed to wait on the next clusters.
377		$masterPositions = array_fill( 0, count( $lbs ), false );
378		foreach ( $lbs as $i => $lb ) {
379		if ( $lb->getServerCount() <= 1 ) {
380		// Bug 27975 - Don't try to wait for slaves if there are none
381		// Prevents permission error when getting master position
382		continue;
383		} elseif ( $opts['ifWritesSince']
384		&& $lb->lastMasterChangeTimestamp() < $opts['ifWritesSince']
385		) {
386		continue; // no writes since the last wait
387		}
388		$masterPositions[$i] = $lb->getMasterPos();
389		}
390
391		$failed = [];
392		foreach ( $lbs as $i => $lb ) {
393		if ( $masterPositions[$i] ) {
394		// The DBMS may not support getMasterPos() or the whole
395		// load balancer might be fake (e.g. $wgAllDBsAreLocalhost).
396		if ( !$lb->waitForAll( $masterPositions[$i], $opts['timeout'] ) ) {
397		$failed[] = $lb->getServerName( $lb->getWriterIndex() );
398		}
399		}
400		}
401
402		if ( $failed ) {
		0 ignored issues – show Bug Best Practice introduced 2016-01-25 18:14 UTC by Report Bug Copy Issue Report The expression `$failed` of type `array` is implicitly converted to a boolean; are you sure this is intended? If so, consider using `! empty($expr)` instead to make it clear that you intend to check for an array without elements. This check marks implicit conversions of arrays to boolean values in a comparison. While in PHP an empty array is considered to be equal (but not identical) to false, this is not always apparent. Consider making the comparison explicit by using `empty(..)` or `! empty(...)` instead. Loading history...
403		throw new DBReplicationWaitError(
404		"Could not wait for slaves to catch up to " .
405		implode( ', ', $failed )
406		);
407		}
408		}
409
410		/**
411		* Disable the ChronologyProtector for all load balancers
412		*
413		* This can be called at the start of special API entry points
414		*
415		* @since 1.27
416		*/
417		public function disableChronologyProtection() {
418		$this->chronProt->setEnabled( false );
419		}
420
421		/**
422		* @return ChronologyProtector
423		*/
424		protected function newChronologyProtector() {
425		$request = RequestContext::getMain()->getRequest();
426		$chronProt = new ChronologyProtector(
427		ObjectCache::getMainStashInstance(),
428		[
429		'ip' => $request->getIP(),
430		'agent' => $request->getHeader( 'User-Agent' )
431		]
432		);
433		if ( PHP_SAPI === 'cli' ) {
434		$chronProt->setEnabled( false );
435		} elseif ( $request->getHeader( 'ChronologyProtection' ) === 'false' ) {
436		// Request opted out of using position wait logic. This is useful for requests
437		// done by the job queue or background ETL that do not have a meaningful session.
438		$chronProt->setWaitEnabled( false );
439		}
440
441		return $chronProt;
442		}
443
444		/**
445		* @param ChronologyProtector $cp
446		*/
447		protected function shutdownChronologyProtector( ChronologyProtector $cp ) {
448		// Get all the master positions needed
449		$this->forEachLB( function ( LoadBalancer $lb ) use ( $cp ) {
450		$cp->shutdownLB( $lb );
451		} );
452		// Write them to the stash
453		$unsavedPositions = $cp->shutdown();
454		// If the positions failed to write to the stash, at least wait on local datacenter
455		// slaves to catch up before responding. Even if there are several DCs, this increases
456		// the chance that the user will see their own changes immediately afterwards. As long
457		// as the sticky DC cookie applies (same domain), this is not even an issue.
458		$this->forEachLB( function ( LoadBalancer $lb ) use ( $unsavedPositions ) {
459		$masterName = $lb->getServerName( $lb->getWriterIndex() );
460		if ( isset( $unsavedPositions[$masterName] ) ) {
461		$lb->waitForAll( $unsavedPositions[$masterName] );
462		}
463		} );
464		}
465		}
466
467		/**
468		* Exception class for attempted DB access
469		*/
470		class DBAccessError extends MWException {
471		public function __construct() {
472		parent::__construct( "Mediawiki tried to access the database via wfGetDB(). " .
473		"This is not allowed." );
474		}
475		}
476
477		/**
478		* Exception class for replica DB wait timeouts
479		*/
480		class DBReplicationWaitError extends Exception {
481		}
482

wikimedia / mediawiki

Branch — master (86dc85)

LBFactory::commitMasterChanges() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like