From: aude <aude.wiki@gmail.com>
Date: Fri, 25 Jan 2013 14:51:23 +0000 (+0000)
Subject: add profiling points in SiteSQLStore
X-Git-Tag: 1.31.0-rc.0~20918^2
X-Git-Url: http://git.cyclocoop.org/%24href?a=commitdiff_plain;h=d514668d5ceda8ac9cd1d59e0e1ef985bd2f8e7d;p=lhc%2Fweb%2Fwiklou.git

add profiling points in SiteSQLStore

Change-Id: I1699baba30c4e0fe5db185410f54d19097cc6eb6
---

diff --git a/includes/site/SiteSQLStore.php b/includes/site/SiteSQLStore.php
index e6087366df..a79c8c5e5d 100644
--- a/includes/site/SiteSQLStore.php
+++ b/includes/site/SiteSQLStore.php
@@ -88,6 +88,8 @@ class SiteSQLStore implements SiteStore {
 	 * @return String The cache key.
 	 */
 	protected function getCacheKey() {
+		wfProfileIn( __METHOD__ );
+
 		if ( $this->cacheKey === null ) {
 			$type = 'SiteList#' . SiteList::getSerialVersionId();
 			$source = $this->sitesTable->getName();
@@ -99,6 +101,7 @@ class SiteSQLStore implements SiteStore {
 			$this->cacheKey = wfMemcKey( "$source/$type" );
 		}
 
+		wfProfileOut( __METHOD__ );
 		return $this->cacheKey;
 	}
 
@@ -112,6 +115,8 @@ class SiteSQLStore implements SiteStore {
 	 * @return SiteList
 	 */
 	public function getSites( $source = 'cache' ) {
+		wfProfileIn( __METHOD__ );
+
 		if ( $source === 'cache' ) {
 			if ( $this->sites === null ) {
 				$cache = wfGetMainCache();
@@ -128,6 +133,7 @@ class SiteSQLStore implements SiteStore {
 			$this->loadSites();
 		}
 
+		wfProfileOut( __METHOD__ );
 		return $this->sites;
 	}
 
@@ -141,6 +147,8 @@ class SiteSQLStore implements SiteStore {
 	 * @return Site
 	 */
 	protected function siteFromRow( ORMRow $siteRow ) {
+		wfProfileIn( __METHOD__ );
+
 		$site = Site::newForType( $siteRow->getField( 'type', Site::TYPE_UNKNOWN ) );
 
 		$site->setGlobalId( $siteRow->getField( 'global_key' ) );
@@ -171,6 +179,7 @@ class SiteSQLStore implements SiteStore {
 			$site->setExtraConfig( $siteRow->getField( 'config' ) );
 		}
 
+		wfProfileOut( __METHOD__ );
 		return $site;
 	}
 
@@ -180,6 +189,8 @@ class SiteSQLStore implements SiteStore {
 	 * @since 1.21
 	 */
 	protected function loadSites() {
+		wfProfileIn( __METHOD__ );
+
 		$this->sites = new SiteList();
 
 		foreach ( $this->sitesTable->select() as $siteRow ) {
@@ -208,6 +219,8 @@ class SiteSQLStore implements SiteStore {
 
 		$cache = wfGetMainCache();
 		$cache->set( $this->getCacheKey(), $this->sites );
+
+		wfProfileOut( __METHOD__ );
 	}
 
 	/**
@@ -221,8 +234,11 @@ class SiteSQLStore implements SiteStore {
 	 * @return Site|null
 	 */
 	public function getSite( $globalId, $source = 'cache' ) {
+		wfProfileIn( __METHOD__ );
+
 		$sites = $this->getSites( $source );
 
+		wfProfileOut( __METHOD__ );
 		return $sites->hasSite( $globalId ) ? $sites->getSite( $globalId ) : null;
 	}
 
@@ -249,7 +265,10 @@ class SiteSQLStore implements SiteStore {
 	 * @return boolean Success indicator
 	 */
 	public function saveSites( array $sites ) {
+		wfProfileIn( __METHOD__ );
+
 		if ( empty( $sites ) ) {
+			wfProfileOut( __METHOD__ );
 			return true;
 		}
 
@@ -325,6 +344,7 @@ class SiteSQLStore implements SiteStore {
 		// purge cache
 		$this->reset();
 
+		wfProfileOut( __METHOD__ );
 		return $success;
 	}
 
@@ -335,10 +355,13 @@ class SiteSQLStore implements SiteStore {
 	 * @since 1.21
 	 */
 	public function reset() {
+		wfProfileIn( __METHOD__ );
 		// purge cache
 		$cache = wfGetMainCache();
 		$cache->delete( $this->getCacheKey() );
 		$this->sites = null;
+
+		wfProfileOut( __METHOD__ );
 	}
 
 	/**
@@ -349,6 +372,7 @@ class SiteSQLStore implements SiteStore {
 	 * @return bool success
 	 */
 	public function clear() {
+		wfProfileIn( __METHOD__ );
 		$dbw = $this->sitesTable->getWriteDbConnection();
 
 		$trx = $dbw->trxLevel();
@@ -366,6 +390,7 @@ class SiteSQLStore implements SiteStore {
 
 		$this->reset();
 
+		wfProfileOut( __METHOD__ );
 		return $ok;
 	}