Addshore has uploaded a new change for review. (
https://gerrit.wikimedia.org/r/394302 )
Change subject: Get page fromdbmaster in Importer
......................................................................
Get page fromdbmaster in Importer
Due to possible slave lag the page might not exist yet.
So get from master.
The 2 selects we now have from master dbs in the Importer
probably hint that we should be doing this in jobs.
Bug: T181391
Change-Id: Ia246302dc0db439dcf25760337a98e169569d828
---
M src/Services/Importer.php
1 file changed, 6 insertions(+), 2 deletions(-)
git pull ssh://gerrit.wikimedia.org:29418/mediawiki/extensions/FileImporter
refs/changes/02/394302/1
diff --git a/src/Services/Importer.php b/src/Services/Importer.php
index acbc775..72024d7 100644
--- a/src/Services/Importer.php
+++ b/src/Services/Importer.php
@@ -147,10 +147,14 @@
User $user
) {
$config = MediaWikiServices::getInstance()->getMainConfig();
- $page = WikiPage::newFromID( $articleIdForUpdate );
+ /**
+ * Pass fromdbmaster as the page has only just been created and
in
+ * multi db setups slaves will have lag.
+ */
+ $page = WikiPage::newFromID( $articleIdForUpdate,
'fromdbmaster' );
if ( $page === null ) {
throw new RuntimeException(
- 'Failed to get wikipedia to create import edit
with id: ' . $articleIdForUpdate
+ 'Failed to get wikipedia to create import edit
with page id: ' . $articleIdForUpdate
);
}
$editResult = $page->doEditContent(
--
To view, visit https://gerrit.wikimedia.org/r/394302
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ia246302dc0db439dcf25760337a98e169569d828
Gerrit-PatchSet: 1
Gerrit-Project: mediawiki/extensions/FileImporter
Gerrit-Branch: master
Gerrit-Owner: Addshore <[email protected]>
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
