[Wikidata-bugs] [Maniphest] [Commented On] T163544: Wikibase\TermSqlIndex::getMatchingTerms , Wikibase\TermSqlIndex::fetchTerms have bad performance after codfw failover

2017-04-23 Thread gerritbot
gerritbot added a comment. Change 349424 merged by jenkins-bot: [mediawiki/extensions/Wikibase@master] Use smaller batches in TermSqlIndex::getTermsOfEntities https://gerrit.wikimedia.org/r/349424TASK DETAILhttps://phabricator.wikimedia.org/T163544EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T163544: Wikibase\TermSqlIndex::getMatchingTerms , Wikibase\TermSqlIndex::fetchTerms have bad performance after codfw failover

2017-04-21 Thread jcrespo
jcrespo added a comment. I was thinking on https://dev.mysql.com/doc/refman/5.6/en/server-system-variables.html#sysvar_eq_range_index_dive_limit but that is probably more wishful thinking than practical give we are on MariaDB 10.TASK DETAILhttps://phabricator.wikimedia.org/T163544EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T163544: Wikibase\TermSqlIndex::getMatchingTerms , Wikibase\TermSqlIndex::fetchTerms have bad performance after codfw failover

2017-04-21 Thread hoo
hoo added a comment. From #wikimedia-databases: jynus: For the first query in https://phabricator.wikimedia.org/T163544 how many term_entity_id would be ok? Right now we don't batch there at all, but I can add that rather easily Would 10 be ok? I would say 9 I thin 10 is the limit to start

[Wikidata-bugs] [Maniphest] [Commented On] T163544: Wikibase\TermSqlIndex::getMatchingTerms , Wikibase\TermSqlIndex::fetchTerms have bad performance after codfw failover

2017-04-21 Thread gerritbot
gerritbot added a comment. Change 349429 merged by jenkins-bot: [mediawiki/extensions/Wikibase@master] Only retrieve up to 2500 terms in TermSqlIndex https://gerrit.wikimedia.org/r/349429TASK DETAILhttps://phabricator.wikimedia.org/T163544EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T163544: Wikibase\TermSqlIndex::getMatchingTerms , Wikibase\TermSqlIndex::fetchTerms have bad performance after codfw failover

2017-04-21 Thread gerritbot
gerritbot added a comment. Change 349429 had a related patch set uploaded (by Hoo man): [mediawiki/extensions/Wikibase@master] Only retrieve up to 2500 terms in TermSqlIndex https://gerrit.wikimedia.org/r/349429TASK DETAILhttps://phabricator.wikimedia.org/T163544EMAIL

[Wikidata-bugs] [Maniphest] [Commented On] T163544: Wikibase\TermSqlIndex::getMatchingTerms , Wikibase\TermSqlIndex::fetchTerms have bad performance after codfw failover

2017-04-21 Thread gerritbot
gerritbot added a comment. Change 349424 had a related patch set uploaded (by Hoo man): [mediawiki/extensions/Wikibase@master] Use smaller batches in TermSqlIndex::getTermsOfEntities https://gerrit.wikimedia.org/r/349424TASK DETAILhttps://phabricator.wikimedia.org/T163544EMAIL