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
F18943811: D19185.diff
Tue, Nov 11, 2:45 PM
F18935263: D19185.diff
Mon, Nov 10, 6:00 PM
F18877689: D19185.id.diff
Thu, Nov 6, 11:24 AM
F18873398: D19185.diff
Wed, Nov 5, 9:32 AM
F18848191: D19185.id45952.diff
Thu, Oct 30, 3:43 AM
F18808378: D19185.id45947.diff
Sun, Oct 19, 6:13 AM
F18737234: D19185.diff
Oct 1 2025, 9:33 AM
F18733756: D19185.id.diff
Sep 30 2025, 10:41 PM
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