Last modified: 2012-04-23 16:49:38 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T34395, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 32395 - Count only links from existing pages on WantedQueryPages
Count only links from existing pages on WantedQueryPages
Status: RESOLVED WONTFIX
Product: MediaWiki
Classification: Unclassified
Special pages (Other open bugs)
1.20.x
All All
: Normal normal (vote)
: ---
Assigned To: Nobody - You can work on this!
: patch, patch-need-review
Depends on:
Blocks:
  Show dependency treegraph
 
Reported: 2011-11-13 16:38 UTC by Umherirrender
Modified: 2012-04-23 16:49 UTC (History)
2 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
changed queries (6.36 KB, patch)
2011-11-13 16:38 UTC, Umherirrender
Details

Description Umherirrender 2011-11-13 16:38:58 UTC
Created attachment 9439 [details]
changed queries

The WantedQueryPages (WantedPages, WantedTemplates, WantedCategories and WantedImages) are showing often ghost entries. Purging the entries is possible (bug 16112) but not down regularly on all wikis.

I have modify the queries of that 4 special page to count only existing pages. Than this ghost entries are gone from the special page output.

To reach this, the query has to join a second time to the page table to check the *_from fields of existing in the page table.

All special pages are expensive. I hope, the change makes the special pages not too expensive and they can stay on wmf wikis.

Feel free to modify the patch.

Thanks.
Comment 1 Sam Reed (reedy) 2011-11-20 15:06:05 UTC
Minor point (not wrong), but as NS_CATEGORY is known to be a number constant, quoting isn't needed, but you're not adding it, just making it database agnostic
Comment 2 Umherirrender 2012-01-19 20:11:10 UTC
In my opinion it is better to set up a cron job to fix this (bug 16112 comment 17), because other query pages also affected by the ghost entries, but due to a low count of that, you will not see the ghost entries in the result of Special:MostLinkedPages (for example).
Comment 3 Umherirrender 2012-04-23 16:49:38 UTC
maintenance script now runs periodically (on wmf), no need for this change, from my point of view.

Feel free to commit the patch, if you think, that it is a good thing, but that can make the queries more expensive.

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links