Aude has uploaded a new change for review.

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

Change subject: dispatchChanges: Don't use FOR UPDATE for selecting a client
......................................................................

dispatchChanges: Don't use FOR UPDATE for selecting a client

Also make the randomness parameter actually work and increase the
default value. In production the value should be even higher.

Change-Id: Ie7d68953be5c680c964d12a4943d72886407d9ad
---
M lib/maintenance/dispatchChanges.php
1 file changed, 5 insertions(+), 5 deletions(-)


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

diff --git a/lib/maintenance/dispatchChanges.php 
b/lib/maintenance/dispatchChanges.php
index f2131d8..c014708 100644
--- a/lib/maintenance/dispatchChanges.php
+++ b/lib/maintenance/dispatchChanges.php
@@ -98,7 +98,7 @@
         *           causes a completely random selection of the target, 
regardless of when it
         *           was last selected for dispatch.
         */
-       protected $randomness = 5;
+       protected $randomness = 10;
 
        /**
         * @var bool: whether output should be version.
@@ -119,7 +119,7 @@
                $this->addOption( 'lock-grace-interval', "Seconds after wich to 
probe for orphaned locks. "
                                        . "Default: 60", false, true );
                $this->addOption( 'randomness', "Number of least current target 
wikis to pick from at random. "
-                                       . "Default: 5.", false, true );
+                                       . "Default: 10.", false, true );
                $this->addOption( 'max-passes', "The number of passes to 
perform. "
                                        . "Default: 1 if --max-time is not set, 
infinite if it is.", false, true );
                $this->addOption( 'max-time', "The number of seconds to run 
before exiting, "
@@ -146,6 +146,7 @@
                $this->delay = intval( $this->getOption( 'idle-delay', 10 ) );
                $this->dispatchInterval = intval( $this->getOption( 
'dispatch-interval', 60 ) );
                $this->lockGraceInterval = intval( $this->getOption( 
'lock-grace-interval', 60 ) );
+               $this->randomness = intval( $this->getOption( 'randomness', 
$this->randomness ) );
 
                $this->verbose = $this->getOption( 'verbose', false );
 
@@ -418,9 +419,8 @@
                        ,
                        __METHOD__,
                        array(
-                               'ORDER BY chd_seen ASC',
-                               'FOR UPDATE',
-                               'LIMIT ' . (int)$this->randomness
+                               'ORDER BY' => 'chd_seen ASC',
+                               'LIMIT' => (int)$this->randomness
                        )
                );
 

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ie7d68953be5c680c964d12a4943d72886407d9ad
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: wmf/1.25wmf22
Gerrit-Owner: Aude <aude.w...@gmail.com>
Gerrit-Reviewer: Hoo man <h...@online.de>

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

Reply via email to