jenkins-bot has submitted this change and it was merged.
Change subject: Use batch inserting of jobs
......................................................................
Use batch inserting of jobs
Change-Id: Iec57820d8814c07004a808270cb28c310cc7ef37
---
M MassMessageSubmitJob.php
1 file changed, 7 insertions(+), 2 deletions(-)
Approvals:
Aaron Schulz: Looks good to me, approved
jenkins-bot: Verified
diff --git a/MassMessageSubmitJob.php b/MassMessageSubmitJob.php
index a50978d..2300934 100644
--- a/MassMessageSubmitJob.php
+++ b/MassMessageSubmitJob.php
@@ -21,10 +21,15 @@
public function run() {
$data = $this->params['data'];
$pages = $this->params['pages'];
+ $jobsByTarget = array();
+
foreach ( $pages as $page ) {
$title = Title::newFromText( $page['title'] );
- $job = new MassMessageJob( $title, $data );
- JobQueueGroup::singleton( $page['wiki'] )->push( $job );
+ $jobsByTarget[$page['wiki']][] = new MassMessageJob(
$title, $data );
+ }
+
+ foreach ( $jobsByTarget as $wiki => $jobs ) {
+ JobQueueGroup::singleton( $wiki )->push( $jobs );
}
return true;
--
To view, visit https://gerrit.wikimedia.org/r/89382
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Iec57820d8814c07004a808270cb28c310cc7ef37
Gerrit-PatchSet: 2
Gerrit-Project: mediawiki/extensions/MassMessage
Gerrit-Branch: master
Gerrit-Owner: Legoktm <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Legoktm <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
