[MediaWiki-commits] [Gerrit] mediawiki...CentralAuth[master]: Skip all large wikis during the maintenance script run

2016-11-29 Thread jenkins-bot (Code Review)
jenkins-bot has submitted this change and it was merged.

Change subject: Skip all large wikis during the maintenance script run
..


Skip all large wikis during the maintenance script run

Change-Id: Ib28e1e7d50e580a2df8ea388ac1b44cdfd130837
---
M maintenance/populateLocalAndGlobalIds.php
1 file changed, 3 insertions(+), 2 deletions(-)

Approvals:
  Kaldari: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/maintenance/populateLocalAndGlobalIds.php 
b/maintenance/populateLocalAndGlobalIds.php
index 69a28be..3fba4ab 100644
--- a/maintenance/populateLocalAndGlobalIds.php
+++ b/maintenance/populateLocalAndGlobalIds.php
@@ -18,8 +18,9 @@
$dbr = CentralAuthUtils::getCentralSlaveDB();
$dbw = CentralAuthUtils::getCentralDB();
foreach( $wgLocalDatabases as $wiki ) {
-   // Temporarily skipping English Wikipedia
-   if ( $wiki == 'enwiki' ) {
+   // Temporarily skipping large wikis, 5 mil seems like a 
safe number (skips en, meta, mediawiki & login wikis)
+   $size = $dbr->estimateRowCount( 'localuser', '*', [ 
'lu_wiki' => $wiki ] );
+   if ( $size > 500 ) {
continue;
}
$lastGlobalId = -1;

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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ib28e1e7d50e580a2df8ea388ac1b44cdfd130837
Gerrit-PatchSet: 3
Gerrit-Project: mediawiki/extensions/CentralAuth
Gerrit-Branch: master
Gerrit-Owner: Niharika29 
Gerrit-Reviewer: Kaldari 
Gerrit-Reviewer: Legoktm 
Gerrit-Reviewer: Reedy 
Gerrit-Reviewer: jenkins-bot <>

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


[MediaWiki-commits] [Gerrit] mediawiki...CentralAuth[master]: Skip all large wikis during the maintenance script run

2016-11-29 Thread Niharika29 (Code Review)
Niharika29 has uploaded a new change for review.

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

Change subject: Skip all large wikis during the maintenance script run
..

Skip all large wikis during the maintenance script run

Change-Id: Ib28e1e7d50e580a2df8ea388ac1b44cdfd130837
---
M maintenance/populateLocalAndGlobalIds.php
1 file changed, 3 insertions(+), 2 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/CentralAuth 
refs/changes/39/324239/1

diff --git a/maintenance/populateLocalAndGlobalIds.php 
b/maintenance/populateLocalAndGlobalIds.php
index 69a28be..b87ad7f 100644
--- a/maintenance/populateLocalAndGlobalIds.php
+++ b/maintenance/populateLocalAndGlobalIds.php
@@ -18,8 +18,9 @@
$dbr = CentralAuthUtils::getCentralSlaveDB();
$dbw = CentralAuthUtils::getCentralDB();
foreach( $wgLocalDatabases as $wiki ) {
-   // Temporarily skipping English Wikipedia
-   if ( $wiki == 'enwiki' ) {
+   // Temporarily skipping large wikis, 5 mil seems like a 
safe number (skips en, meta, mediawiki & login wikis)
+   $size = $dbr->estimateRowCount( 'localuser', '*', 
'lu_wiki = $wiki' );
+   if ( $size > 500 ) {
continue;
}
$lastGlobalId = -1;

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ib28e1e7d50e580a2df8ea388ac1b44cdfd130837
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/CentralAuth
Gerrit-Branch: master
Gerrit-Owner: Niharika29 

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