Page MenuHomePhabricator

Optimize feed story gc query.
AbandonedPublic

Authored by Pawka on Sep 10 2015, 11:34 AM.
Tags
None
Referenced Files
F15526233: D14091.id34058.diff
Mon, Apr 21, 8:10 PM
F15518058: D14091.id.diff
Sat, Apr 19, 10:35 AM
F15516236: D14091.diff
Fri, Apr 18, 5:30 PM
F15511423: D14091.id34058.diff
Thu, Apr 17, 2:24 AM
F15511422: D14091.id.diff
Thu, Apr 17, 2:24 AM
F15511421: D14091.diff
Thu, Apr 17, 2:24 AM
F15437654: D14091.diff
Mar 25 2025, 8:51 PM
F15433217: D14091.id.diff
Mar 24 2025, 10:06 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

Ref: T9382. Removed ORDER BY clause to make query perform faster.

Test Plan

Launch phd workers and check if older than 90 days feed stories are
removed.

Diff Detail

Event Timeline

Pawka retitled this revision from to Optimize feed story gc query..
Pawka updated this object.
Pawka edited the test plan for this revision. (Show Details)
Pawka added a reviewer: epriestley.

See T9382#136333. I'd like to understand this behavior better before making this change.

Notably, this makes the query plan worse on our dataset, although we already GC'd so we don't have a large dataset handy to test if it makes the actual query worse or not.

Let's ignore DIFF this then. Probably this is some configuration issue what we do not see now ;)