Aaron Schulz has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/311380

Change subject: Move more LockManager classes to /libs
......................................................................

Move more LockManager classes to /libs

Also removed unused getLocalLB() method in MySqlLockManager.

Change-Id: I4bfa79f430827b8717a57bb61cb7c36bc7fcb489
---
M autoload.php
M includes/filebackend/lockmanager/MySqlLockManager.php
R includes/libs/lockmanager/DBLockManager.php
R includes/libs/lockmanager/PostgreSqlLockManager.php
R includes/libs/lockmanager/ScopedLock.php
5 files changed, 3 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/core 
refs/changes/80/311380/1

diff --git a/autoload.php b/autoload.php
index ff7d488..ed4e8c9 100644
--- a/autoload.php
+++ b/autoload.php
@@ -306,7 +306,7 @@
        'DBError' => __DIR__ . '/includes/libs/rdbms/exception/DBError.php',
        'DBExpectedError' => __DIR__ . 
'/includes/libs/rdbms/exception/DBError.php',
        'DBFileJournal' => __DIR__ . 
'/includes/filebackend/filejournal/DBFileJournal.php',
-       'DBLockManager' => __DIR__ . 
'/includes/filebackend/lockmanager/DBLockManager.php',
+       'DBLockManager' => __DIR__ . 
'/includes/libs/lockmanager/DBLockManager.php',
        'DBMasterPos' => __DIR__ . 
'/includes/libs/rdbms/database/position/DBMasterPos.php',
        'DBQueryError' => __DIR__ . 
'/includes/libs/rdbms/exception/DBError.php',
        'DBReadOnlyError' => __DIR__ . 
'/includes/libs/rdbms/exception/DBError.php',
@@ -1069,7 +1069,7 @@
        'PopulateRecentChangesSource' => __DIR__ . 
'/maintenance/populateRecentChangesSource.php',
        'PopulateRevisionLength' => __DIR__ . 
'/maintenance/populateRevisionLength.php',
        'PopulateRevisionSha1' => __DIR__ . 
'/maintenance/populateRevisionSha1.php',
-       'PostgreSqlLockManager' => __DIR__ . 
'/includes/filebackend/lockmanager/PostgreSqlLockManager.php',
+       'PostgreSqlLockManager' => __DIR__ . 
'/includes/libs/lockmanager/PostgreSqlLockManager.php',
        'PostgresBlob' => __DIR__ . 
'/includes/libs/rdbms/encasing/PostgresBlob.php',
        'PostgresField' => __DIR__ . '/includes/db/DatabasePostgres.php',
        'PostgresInstaller' => __DIR__ . 
'/includes/installer/PostgresInstaller.php',
@@ -1228,7 +1228,7 @@
        'Sanitizer' => __DIR__ . '/includes/Sanitizer.php',
        'SavepointPostgres' => __DIR__ . '/includes/db/DatabasePostgres.php',
        'ScopedCallback' => __DIR__ . '/includes/libs/ScopedCallback.php',
-       'ScopedLock' => __DIR__ . 
'/includes/filebackend/lockmanager/ScopedLock.php',
+       'ScopedLock' => __DIR__ . '/includes/libs/lockmanager/ScopedLock.php',
        'SearchApi' => __DIR__ . '/includes/api/SearchApi.php',
        'SearchDatabase' => __DIR__ . '/includes/search/SearchDatabase.php',
        'SearchDump' => __DIR__ . '/maintenance/dumpIterator.php',
diff --git a/includes/filebackend/lockmanager/MySqlLockManager.php 
b/includes/filebackend/lockmanager/MySqlLockManager.php
index 124d410..441ffea 100644
--- a/includes/filebackend/lockmanager/MySqlLockManager.php
+++ b/includes/filebackend/lockmanager/MySqlLockManager.php
@@ -15,11 +15,6 @@
                self::LOCK_EX => self::LOCK_EX
        ];
 
-       protected function getLocalLB() {
-               // Use a separate connection so releaseAllLocks() doesn't 
rollback the main trx
-               return wfGetLBFactory()->newMainLB( $this->domain );
-       }
-
        protected function initConnection( $lockDb, IDatabase $db ) {
                # Let this transaction see lock rows from other transactions
                $db->query( "SET SESSION TRANSACTION ISOLATION LEVEL READ 
UNCOMMITTED;" );
diff --git a/includes/filebackend/lockmanager/DBLockManager.php 
b/includes/libs/lockmanager/DBLockManager.php
similarity index 100%
rename from includes/filebackend/lockmanager/DBLockManager.php
rename to includes/libs/lockmanager/DBLockManager.php
diff --git a/includes/filebackend/lockmanager/PostgreSqlLockManager.php 
b/includes/libs/lockmanager/PostgreSqlLockManager.php
similarity index 100%
rename from includes/filebackend/lockmanager/PostgreSqlLockManager.php
rename to includes/libs/lockmanager/PostgreSqlLockManager.php
diff --git a/includes/filebackend/lockmanager/ScopedLock.php 
b/includes/libs/lockmanager/ScopedLock.php
similarity index 100%
rename from includes/filebackend/lockmanager/ScopedLock.php
rename to includes/libs/lockmanager/ScopedLock.php

-- 
To view, visit https://gerrit.wikimedia.org/r/311380
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I4bfa79f430827b8717a57bb61cb7c36bc7fcb489
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: master
Gerrit-Owner: Aaron Schulz <asch...@wikimedia.org>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to