Aude has uploaded a new change for review.

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

Change subject: Wait for slaves in BulkSubscriptionUpdater
......................................................................

Wait for slaves in BulkSubscriptionUpdater

especially needed for the purge subscriptions option

Change-Id: I0babedaa0fb0cef1e1cf1ae5c526854d078da7bd
---
M client/includes/store/sql/BulkSubscriptionUpdater.php
1 file changed, 4 insertions(+), 0 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/Wikibase 
refs/changes/60/208860/1

diff --git a/client/includes/store/sql/BulkSubscriptionUpdater.php 
b/client/includes/store/sql/BulkSubscriptionUpdater.php
index c85fb93..bf964a0 100644
--- a/client/includes/store/sql/BulkSubscriptionUpdater.php
+++ b/client/includes/store/sql/BulkSubscriptionUpdater.php
@@ -124,6 +124,8 @@
                $continuation = $startEntity === null ? null : array( 
$startEntity->getSerialization() );
 
                while ( true ) {
+                       wfWaitForSlaves();
+
                        $count = $this->processUpdateBatch( $continuation );
 
                        if ( $count > 0 ) {
@@ -263,6 +265,8 @@
                $this->repoConnectionManager->forceMaster();
 
                while ( true ) {
+                       wfWaitForSlaves();
+
                        $count = $this->processDeletionBatch( $continuation );
 
                        if ( $count > 0 ) {

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I0babedaa0fb0cef1e1cf1ae5c526854d078da7bd
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Aude <aude.w...@gmail.com>

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

Reply via email to