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
F15564428: D19185.id.diff
Wed, Apr 30, 12:10 PM
F15561285: D19185.diff
Tue, Apr 29, 7:01 PM
F15559581: D19185.diff
Tue, Apr 29, 9:32 AM
F15457740: D19185.id45947.diff
Mar 30 2025, 6:22 PM
F15456331: D19185.id45952.diff
Mar 30 2025, 8:46 AM
F15453568: D19185.id.diff
Mar 29 2025, 12:41 PM
F15449529: D19185.diff
Mar 28 2025, 10:48 AM
F15441374: D19185.id45947.diff
Mar 26 2025, 5:51 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