jenkins-bot has submitted this change and it was merged.
Change subject: The "masterPos" field in refreshLinks can be false so check for
that
......................................................................
The "masterPos" field in refreshLinks can be false so check for that
Change-Id: I445e10da122e5f24e589f1eb078a31605b9f2e13
(cherry picked from commit 653f6c80d71b8d69ffe1838cd235096a82bbda42)
---
M includes/job/jobs/RefreshLinksJob.php
1 file changed, 1 insertion(+), 1 deletion(-)
Approvals:
Aaron Schulz: Looks good to me, approved
jenkins-bot: Verified
diff --git a/includes/job/jobs/RefreshLinksJob.php
b/includes/job/jobs/RefreshLinksJob.php
index 563ce2f..474212b 100644
--- a/includes/job/jobs/RefreshLinksJob.php
+++ b/includes/job/jobs/RefreshLinksJob.php
@@ -48,7 +48,7 @@
# Wait for the DB of the current/next slave DB handle to catch
up to the master.
# This way, we get the correct page_latest for templates or
files that just changed
# milliseconds ago, having triggered this job to begin with.
- if ( isset( $this->params['masterPos'] ) ) {
+ if ( !empty( $this->params['masterPos'] ) ) {
wfGetLB()->waitFor( $this->params['masterPos'] );
}
--
To view, visit https://gerrit.wikimedia.org/r/80498
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I445e10da122e5f24e589f1eb078a31605b9f2e13
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/core
Gerrit-Branch: wmf/1.22wmf13
Gerrit-Owner: Aaron Schulz <[email protected]>
Gerrit-Reviewer: Aaron Schulz <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
