Page MenuHomePhabricator

Trending shows same article twice
Closed, ResolvedPublic

Description

Screenshot_20180523-211455.png (1×1 px, 329 KB)

See attached screenshot. "Megan, Duchess of Sussex" article appears twice in the Trending feed. I would expect the same article to appear just once. App version: 2.7.232-r-2018-04-17

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald Transcript

Thanks for the bug report. I noticed this myself just yesterday. Will look into it.

Tagging the iOS app too since I noticed it in the iOS app. This is probably just for tracking for the iOS team, though.

I think this is sort of a one-off situation caused by the recent move of the "Meghan Markle" page to "Meghan, Duchess of Sussex" (by Jimbo Wales himself). We pick up both separately in our pageview data (example), and the redirect is resolved during processing. I guess we'll need to add a dedupe step or something.

Change 434806 had a related patch set uploaded (by Mholloway; owner: Mholloway):
[mediawiki/services/mobileapps@master] Most-read: Filter redirect-caused duplicates

https://gerrit.wikimedia.org/r/434806

Triaging this as low-priority since the situation in which it arises—in which an article is moved precisely as it spikes in popularity—is relatively rare.

Vvjjkkii renamed this task from Trending shows same article twice to tfcaaaaaaa.Jul 1 2018, 1:08 AM
Vvjjkkii removed Mholloway as the assignee of this task.
Vvjjkkii raised the priority of this task from Low to High.
Vvjjkkii updated the task description. (Show Details)
Vvjjkkii removed subscribers: gerritbot, Aklapper.
Kaartic renamed this task from tfcaaaaaaa to Trending shows same article twice.Jul 1 2018, 3:05 PM
Kaartic assigned this task to Mholloway.
Kaartic lowered the priority of this task from High to Low.
Kaartic updated the task description. (Show Details)
Kaartic added subscribers: Aklapper, gerritbot.

Change 434806 abandoned by Mholloway:
Most-read: Filter redirect-caused duplicates

Reason:
After some discussion, it seems we'll be going in the direction of generalizing result deduplication in RESTBase.

https://gerrit.wikimedia.org/r/434806

Stashbot added a subscriber: Stashbot.

Mentioned in SAL (#wikimedia-operations) [2018-07-17T10:31:32Z] <mobrovac@deploy1001> Started deploy [restbase/deploy@622941d]: Expose the data/mobile/javascript end point, deduplicate most-read results and increase page/related response size to 20 - T199458 T195390

Mentioned in SAL (#wikimedia-operations) [2018-07-17T10:43:36Z] <mobrovac@deploy1001> Started deploy [restbase/deploy@622941d]: Expose the data/mobile/javascript end point, deduplicate most-read results and increase page/related response size to 20, take #2 - T199458 T195390

Mentioned in SAL (#wikimedia-operations) [2018-07-17T11:03:19Z] <mobrovac@deploy1001> Finished deploy [restbase/deploy@622941d]: Expose the data/mobile/javascript end point, deduplicate most-read results and increase page/related response size to 20, take #2 - T199458 T195390 (duration: 19m 43s)

mobrovac edited projects, added Services (done), RESTBase-API; removed Patch-For-Review.
mobrovac added a subscriber: mobrovac.

Deployed, resolving.