X-Git-Url: https://git.heureux-cyclage.org/?a=blobdiff_plain;f=includes%2FExternalStoreDB.php;h=7b4ffc2fa6086d31b3c65339f36f8367d75f9ace;hb=a2b6b9ab699cfe0614051a73ea446ca5ff88e290;hp=d07e23b4d10aa486a65a17a6c856fce32d3efcfe;hpb=41d6bf368bda2d6c86ae4aadffb3428460b3e03c;p=lhc%2Fweb%2Fwiklou.git

diff --git a/includes/ExternalStoreDB.php b/includes/ExternalStoreDB.php
index d07e23b4d1..7b4ffc2fa6 100644
--- a/includes/ExternalStoreDB.php
+++ b/includes/ExternalStoreDB.php
@@ -1,15 +1,12 @@
 <?php
 /**
- * 
- * @package MediaWiki
+ *
  *
  * DB accessable external objects
  *
  */
-require_once( 'LoadBalancer.php' ); 
 
 
-/** @package MediaWiki */
 
 /**
  * External database storage will use one (or more) separate connection pools
@@ -32,34 +29,43 @@ global $wgExternalBlobCache;
 $wgExternalBlobCache = array();
 
 class ExternalStoreDB {
-	/**
-	 * Fetch data from given URL
-	 * @param string $url An url
-	 */
 
+	/** @todo Document.*/
 	function &getLoadBalancer( $cluster ) {
 		global $wgExternalServers, $wgExternalLoadBalancers;
 		if ( !array_key_exists( $cluster, $wgExternalLoadBalancers ) ) {
 			$wgExternalLoadBalancers[$cluster] = LoadBalancer::newFromParams( $wgExternalServers[$cluster] );
 		}
+		$wgExternalLoadBalancers[$cluster]->allowLagged(true);
 		return $wgExternalLoadBalancers[$cluster];
 	}
-	
+
+	/** @todo Document.*/
 	function &getSlave( $cluster ) {
 		$lb =& $this->getLoadBalancer( $cluster );
 		return $lb->getConnection( DB_SLAVE );
 	}
 
+	/** @todo Document.*/
 	function &getMaster( $cluster ) {
 		$lb =& $this->getLoadBalancer( $cluster );
 		return $lb->getConnection( DB_MASTER );
-	}		
-	
+	}
+
+	/** @todo Document.*/
+	function getTable( &$db ) {
+		$table = $db->getLBInfo( 'blobs table' );
+		if ( is_null( $table ) ) {
+			$table = 'blobs';
+		}
+		return $table;
+	}
+
+	/**
+	 * Fetch data from given URL
+	 * @param string $url An url of the form DB://cluster/id or DB://cluster/id/itemid for concatened storage.
+	 */
 	function fetchFromURL($url) {
-		global $wgExternalServers;
-		#
-		# URLs have the form DB://cluster/id or DB://cluster/id/itemid for concatenated storage
-		#
 		$path = explode( '/', $url );
 		$cluster  = $path[2];
 		$id	  = $path[3];
@@ -71,18 +77,21 @@ class ExternalStoreDB {
 
 		$ret =& $this->fetchBlob( $cluster, $id, $itemID );
 
-		if ( $itemID !== false ) {
+		if ( $itemID !== false && $ret !== false ) {
 			return $ret->getItem( $itemID );
 		}
 		return $ret;
 	}
-	
+
 	/**
 	 * Fetch a blob item out of the database; a cache of the last-loaded
 	 * blob will be kept so that multiple loads out of a multi-item blob
 	 * can avoid redundant database access and decompression.
+	 * @param $cluster
+	 * @param $id
+	 * @param $itemID
 	 * @return mixed
-	 * @access private
+	 * @private
 	 */
 	function &fetchBlob( $cluster, $id, $itemID ) {
 		global $wgExternalBlobCache;
@@ -91,16 +100,25 @@ class ExternalStoreDB {
 			wfDebug( "ExternalStoreDB::fetchBlob cache hit on $cacheID\n" );
 			return $wgExternalBlobCache[$cacheID];
 		}
-		
+
 		wfDebug( "ExternalStoreDB::fetchBlob cache miss on $cacheID\n" );
-		
+
 		$dbr =& $this->getSlave( $cluster );
-		$ret = $dbr->selectField( 'blobs', 'blob_text', array( 'blob_id' => $id ) );
-		if( $itemID !== false ) {
+		$ret = $dbr->selectField( $this->getTable( $dbr ), 'blob_text', array( 'blob_id' => $id ) );
+		if ( $ret === false ) {
+			wfDebugLog( 'ExternalStoreDB', "ExternalStoreDB::fetchBlob master fallback on $cacheID\n" );
+			// Try the master
+			$dbw =& $this->getMaster( $cluster );
+			$ret = $dbw->selectField( $this->getTable( $dbw ), 'blob_text', array( 'blob_id' => $id ) );
+			if( $ret === false) {
+				wfDebugLog( 'ExternalStoreDB', "ExternalStoreDB::fetchBlob master failed to find $cacheID\n" );
+			}
+		}
+		if( $itemID !== false && $ret !== false ) {
 			// Unserialise object; caller extracts item
 			$ret = unserialize( $ret );
 		}
-		
+
 		$wgExternalBlobCache = array( $cacheID => &$ret );
 		return $ret;
 	}
@@ -108,19 +126,22 @@ class ExternalStoreDB {
 	/**
 	 * Insert a data item into a given cluster
 	 *
-	 * @param string $cluster The cluster name
-	 * @param string $data The data item
+	 * @param $cluster String: the cluster name
+	 * @param $data String: the data item
 	 * @return string URL
 	 */
 	function store( $cluster, $data ) {
-		global $wgExternalServers;
 		$fname = 'ExternalStoreDB::store';
 
 		$dbw =& $this->getMaster( $cluster );
 
 		$id = $dbw->nextSequenceValue( 'blob_blob_id_seq' );
-		$dbw->insert( 'blobs', array( 'blob_id' => $id, 'blob_text' => $data ), $fname );
-		return "DB://$cluster/" . $dbw->insertId();
+		$dbw->insert( $this->getTable( $dbw ), array( 'blob_id' => $id, 'blob_text' => $data ), $fname );
+		$id = $dbw->insertId();
+		if ( $dbw->getFlag( DBO_TRX ) ) {
+			$dbw->immediateCommit();
+		}
+		return "DB://$cluster/$id";
 	}
 }
 ?>