jenkins-bot has submitted this change and it was merged.

Change subject: Use lazyPush() instead of push() in a few places for jobs
......................................................................


Use lazyPush() instead of push() in a few places for jobs

Change-Id: I56661ea29988e4ee217a63f4c3ffcb78333cc454
(cherry picked from commit a17cdcbf5bac5453a7127482761bb3a8129de2ad)
---
M includes/deferred/HTMLCacheUpdate.php
M includes/mail/EmailNotification.php
M includes/page/WikiPage.php
3 files changed, 3 insertions(+), 3 deletions(-)

Approvals:
  Ori.livneh: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/includes/deferred/HTMLCacheUpdate.php 
b/includes/deferred/HTMLCacheUpdate.php
index 79a10e6..20e4a4c 100644
--- a/includes/deferred/HTMLCacheUpdate.php
+++ b/includes/deferred/HTMLCacheUpdate.php
@@ -55,7 +55,7 @@
 
                $count = $this->mTitle->getBacklinkCache()->getNumLinks( 
$this->mTable, 100 );
                if ( $count >= 100 ) { // many backlinks
-                       JobQueueGroup::singleton()->push( $job );
+                       JobQueueGroup::singleton()->lazyPush( $job );
                        JobQueueGroup::singleton()->deduplicateRootJob( $job );
                } else { // few backlinks ($count might be off even if 0)
                        $dbw = wfGetDB( DB_MASTER );
diff --git a/includes/mail/EmailNotification.php 
b/includes/mail/EmailNotification.php
index 81c4e38..5ed52c4 100644
--- a/includes/mail/EmailNotification.php
+++ b/includes/mail/EmailNotification.php
@@ -168,7 +168,7 @@
                                'pageStatus' => $pageStatus
                        );
                        $job = new EnotifNotifyJob( $title, $params );
-                       JobQueueGroup::singleton()->push( $job );
+                       JobQueueGroup::singleton()->lazyPush( $job );
                } else {
                        $this->actuallyNotifyOnPageChange(
                                $editor,
diff --git a/includes/page/WikiPage.php b/includes/page/WikiPage.php
index 68ae096..03d908c 100644
--- a/includes/page/WikiPage.php
+++ b/includes/page/WikiPage.php
@@ -2213,7 +2213,7 @@
                                $jobs[] = RecentChangesUpdateJob::newPurgeJob();
                        }
 
-                       JobQueueGroup::singleton()->push( $jobs );
+                       JobQueueGroup::singleton()->lazyPush( $jobs );
                }
 
                if ( !$this->exists() ) {

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

Gerrit-MessageType: merged
Gerrit-Change-Id: I56661ea29988e4ee217a63f4c3ffcb78333cc454
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.26wmf6
Gerrit-Owner: Aaron Schulz <asch...@wikimedia.org>
Gerrit-Reviewer: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

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

Reply via email to