[Wikidata-bugs] [Maniphest] [Commented On] T153076: InvalidArgumentException: "Could not load data from the database of repository" in invalid wbgetentities calls
WMDE-leszek added a comment. Silly question, why is it asking for ids=PROPERTY:P120 but not ids=P120. While working on federation we decided that using a prefix with a colon would be used to state what repository the entity belongs to. So while asking the API for PROPERTY:P120 lookups tries to access repo PROPERTY which is obviously not configured etc. Did we miss some important usage here and should not be making such assumption?TASK DETAILhttps://phabricator.wikimedia.org/T153076EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszekCc: Aklapper, thiemowmde, aude, WMDE-leszek, daniel, hoo, D3r1ck01, Izno, Wikidata-bugs, Mbch331, Jay8g, Krenair___ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
[Wikidata-bugs] [Maniphest] [Commented On] T153076: InvalidArgumentException: "Could not load data from the database of repository" in invalid wbgetentities calls
WMDE-leszek added a comment. Agreed. I'll try to fix the code using the lookup so it handles the exception.TASK DETAILhttps://phabricator.wikimedia.org/T153076EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszekCc: TerraCodes, Lydia_Pintscher, Aklapper, thiemowmde, aude, WMDE-leszek, daniel, hoo, JakeTheDeveloper, D3r1ck01, MuhammadShuaib, Izno, Wikidata-bugs, Mbch331, Jay8g, Krenair___ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
[Wikidata-bugs] [Maniphest] [Commented On] T153076: InvalidArgumentException: "Could not load data from the database of repository" in invalid wbgetentities calls
gerritbot added a comment. Change 326970 had a related patch set uploaded (by WMDE-leszek): Make API wbgetentities skip entities from unknown repositories https://gerrit.wikimedia.org/r/326970TASK DETAILhttps://phabricator.wikimedia.org/T153076EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszek, gerritbotCc: gerritbot, TerraCodes, Lydia_Pintscher, Aklapper, thiemowmde, aude, WMDE-leszek, daniel, hoo, JakeTheDeveloper, D3r1ck01, MuhammadShuaib, Izno, Wikidata-bugs, Mbch331, Jay8g, Krenair___ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
[Wikidata-bugs] [Maniphest] [Commented On] T153076: InvalidArgumentException: "Could not load data from the database of repository" in invalid wbgetentities calls
daniel added a comment. @WMDE-leszek regarding Property:P123: that's the page title of P123. Using page titles instead of entity IDs is a common mistake. In the past, IDs could not contain a oclon, so they would just bounce as malformed. Now they look like foreign IDs from an unknown repo, so they hit a different code path, and fail in a different way. We should indeed make sure that this is handled gracefully. Btw, I hope nobody gets the idea of using "property" as a repo prefix repo prefixes and namespaces should not conflict. But this is really no different from MediaWiki's normal interwiki prefixes. They also compete with wiki namespaces as title prefixes.TASK DETAILhttps://phabricator.wikimedia.org/T153076EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszek, danielCc: gerritbot, TerraCodes, Lydia_Pintscher, Aklapper, thiemowmde, aude, WMDE-leszek, daniel, hoo, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, Lewizho99, Maathavan, D3r1ck01, MuhammadShuaib, Izno, Wikidata-bugs, Mbch331, Jay8g, Krenair___ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
[Wikidata-bugs] [Maniphest] [Commented On] T153076: InvalidArgumentException: "Could not load data from the database of repository" in invalid wbgetentities calls
gerritbot added a comment. Change 328488 had a related patch set uploaded (by WMDE-leszek): Skip invalid entity ids in PrefetchingWikiPageEntityMetaDataAccessor https://gerrit.wikimedia.org/r/328488TASK DETAILhttps://phabricator.wikimedia.org/T153076EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszek, gerritbotCc: gerritbot, TerraCodes, Lydia_Pintscher, Aklapper, thiemowmde, aude, WMDE-leszek, daniel, hoo, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, Lewizho99, Maathavan, D3r1ck01, MuhammadShuaib, Izno, Wikidata-bugs, Mbch331, Jay8g, Krenair___ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
[Wikidata-bugs] [Maniphest] [Commented On] T153076: InvalidArgumentException: "Could not load data from the database of repository" in invalid wbgetentities calls
gerritbot added a comment. Change 328488 merged by jenkins-bot: Skip invalid entity ids in PrefetchingWikiPageEntityMetaDataAccessor https://gerrit.wikimedia.org/r/328488TASK DETAILhttps://phabricator.wikimedia.org/T153076EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszek, gerritbotCc: gerritbot, TerraCodes, Lydia_Pintscher, Aklapper, thiemowmde, aude, WMDE-leszek, daniel, hoo, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, Lewizho99, Maathavan, D3r1ck01, MuhammadShuaib, Izno, Wikidata-bugs, Mbch331, Jay8g, Krenair___ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs
[Wikidata-bugs] [Maniphest] [Commented On] T153076: InvalidArgumentException: "Could not load data from the database of repository" in invalid wbgetentities calls
gerritbot added a comment. Change 326970 merged by jenkins-bot: Make API wbgetentities skip entities from unknown repositories https://gerrit.wikimedia.org/r/326970TASK DETAILhttps://phabricator.wikimedia.org/T153076EMAIL PREFERENCEShttps://phabricator.wikimedia.org/settings/panel/emailpreferences/To: WMDE-leszek, gerritbotCc: gerritbot, TerraCodes, Lydia_Pintscher, Aklapper, thiemowmde, aude, WMDE-leszek, daniel, hoo, Th3d3v1ls, JakeTheDeveloper, Ramalepe, Liugev6, Lewizho99, Maathavan, D3r1ck01, MuhammadShuaib, Izno, Wikidata-bugs, Mbch331, Jay8g, Krenair___ Wikidata-bugs mailing list Wikidata-bugs@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/wikidata-bugs