jenkins-bot has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/356081 )
Change subject: SiteStats::jobs fix when there is a single job ...................................................................... SiteStats::jobs fix when there is a single job wfSpecialStatistics() used a database to store the jobs. When estimating the number of row in the `job` table, even when there is no row MySQL would still do a single row read. Hence it erroneously returned 1. That got fixed in 7cd0b2e0b (T12228) with a hack. Nowadays, the functionality has been moved to SiteStats which is backed up by a JobQueueGroup. It now returns the exact number of jobs and the hack is no more needed. We failed to remove it when porting it to the JobQueue system. Change-Id: I6ef55b3dcff3e47a92e2eeeddcbae078654f815a --- M includes/SiteStats.php 1 file changed, 0 insertions(+), 7 deletions(-) Approvals: Krinkle: Looks good to me, approved jenkins-bot: Verified diff --git a/includes/SiteStats.php b/includes/SiteStats.php index 6ce1aed..86a4f63 100644 --- a/includes/SiteStats.php +++ b/includes/SiteStats.php @@ -222,13 +222,6 @@ } catch ( JobQueueError $e ) { self::$jobs = 0; } - /** - * Zero rows still do single row read for row that doesn't exist, - * but people are annoyed by that - */ - if ( self::$jobs == 1 ) { - self::$jobs = 0; - } } return self::$jobs; } -- To view, visit https://gerrit.wikimedia.org/r/356081 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I6ef55b3dcff3e47a92e2eeeddcbae078654f815a Gerrit-PatchSet: 1 Gerrit-Project: mediawiki/core Gerrit-Branch: master Gerrit-Owner: Hashar <has...@free.fr> Gerrit-Reviewer: Aaron Schulz <asch...@wikimedia.org> Gerrit-Reviewer: Addshore <addshorew...@gmail.com> Gerrit-Reviewer: Krinkle <krinklem...@gmail.com> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list MediaWiki-commits@lists.wikimedia.org https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits