Page MenuHomePhabricator

Use "pathIndex" in some owners package queries to improve query plans
ClosedPublic

Authored by epriestley on Mar 7 2018, 4:28 AM.
Tags
None
Referenced Files
F19921356: D19185.diff
Mon, Apr 6, 4:16 AM
F19914803: D19185.id45947.diff
Fri, Apr 3, 2:30 PM
F19898866: D19185.id45947.diff
Sun, Mar 29, 4:23 PM
F19890947: D19185.diff
Thu, Mar 26, 10:23 PM
F19830284: D19185.id45952.diff
Mar 10 2026, 11:59 AM
F19814725: D19185.diff
Mar 4 2026, 3:21 PM
F19563158: D19185.diff
Jan 31 2026, 3:17 PM
F19559569: D19185.id45947.diff
Jan 31 2026, 1:56 AM
Subscribers
None

Details

Summary

Depends on D19184. Ref T11015. Now that we have a digest index column, we can improve some of the queries a bit.

Test Plan
  • Ran queries from revision pages before and after with and without EXPLAIN.
  • Saw the same results with much better EXPLAIN plans.
  • Fragment size is now fixed at 12 bytes per fragment, so we can shove more of them in a single query.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable