jenkins-bot has submitted this change and it was merged. (
https://gerrit.wikimedia.org/r/383646 )
Change subject: Only wait for the client wiki's DB in InjectRCRecordsJob
......................................................................
Only wait for the client wiki's DB in InjectRCRecordsJob
No need to wait for all present LBs. In production this currently
causes this job to always also wait for replication on s5, which
we don't write to here.
Change-Id: I3035e27dbedef9b8c90c41e2d33af61660f82e79
---
M client/includes/Changes/InjectRCRecordsJob.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Ladsgroup: Looks good to me, approved
jenkins-bot: Verified
diff --git a/client/includes/Changes/InjectRCRecordsJob.php
b/client/includes/Changes/InjectRCRecordsJob.php
index b374f41..fc14a0d 100644
--- a/client/includes/Changes/InjectRCRecordsJob.php
+++ b/client/includes/Changes/InjectRCRecordsJob.php
@@ -273,7 +273,7 @@
}
}
- $this->lbFactory->commitAndWaitForReplication( __METHOD__,
$trxToken );
+ $this->lbFactory->commitAndWaitForReplication( __METHOD__,
$trxToken, [ 'domain' => wfWikiID() ] );
$this->incrementStats( 'InjectRCRecords.run.titles', count(
$titles ) );
--
To view, visit https://gerrit.wikimedia.org/r/383646
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I3035e27dbedef9b8c90c41e2d33af61660f82e79
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/Wikibase
Gerrit-Branch: master
Gerrit-Owner: Hoo man <[email protected]>
Gerrit-Reviewer: Aude <[email protected]>
Gerrit-Reviewer: Daniel Kinzler <[email protected]>
Gerrit-Reviewer: Ladsgroup <[email protected]>
Gerrit-Reviewer: Thiemo Mättig (WMDE) <[email protected]>
Gerrit-Reviewer: jenkins-bot <>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
