Ppchelko has uploaded a new change for review. ( 
https://gerrit.wikimedia.org/r/393750 )

Change subject: Stop using EnqueueJob for AddUsagesForPageJob
......................................................................

Stop using EnqueueJob for AddUsagesForPageJob

Since the job was tranferred to kafka-based infrastructure,
no need to wrap it in the EnqueueJob any more. This will lower
the job execution latency and lower the load on the queue.

Bug: T181216
Change-Id: I91187badd680763383e0f26ee1689b9fc01e57fe
---
M client/includes/Hooks/DataUpdateHookHandlers.php
1 file changed, 1 insertion(+), 3 deletions(-)


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

diff --git a/client/includes/Hooks/DataUpdateHookHandlers.php 
b/client/includes/Hooks/DataUpdateHookHandlers.php
index 5b49574..4e00211 100644
--- a/client/includes/Hooks/DataUpdateHookHandlers.php
+++ b/client/includes/Hooks/DataUpdateHookHandlers.php
@@ -165,9 +165,7 @@
                //TODO: Before posting a job, check replica database. If no 
changes are needed, skip update.
 
                $addUsagesForPageJob = AddUsagesForPageJob::newSpec( $title, 
$usageAcc->getUsages() );
-               $enqueueJob = EnqueueJob::newFromLocalJobs( 
$addUsagesForPageJob );
-
-               $this->jobScheduler->lazyPush( $enqueueJob );
+               $this->jobScheduler->lazyPush( $addUsagesForPageJob );
        }
 
        /**

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: I91187badd680763383e0f26ee1689b9fc01e57fe
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Ppchelko <ppche...@wikimedia.org>

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

Reply via email to