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
F15457740: D19185.id45947.diff
Sun, Mar 30, 6:22 PM
F15456331: D19185.id45952.diff
Sun, Mar 30, 8:46 AM
F15453568: D19185.id.diff
Sat, Mar 29, 12:41 PM
F15449529: D19185.diff
Fri, Mar 28, 10:48 AM
F15441374: D19185.id45947.diff
Wed, Mar 26, 5:51 PM
F15441373: D19185.id.diff
Wed, Mar 26, 5:51 PM
F15441372: D19185.id45952.diff
Wed, Mar 26, 5:51 PM
F15441229: D19185.diff
Wed, Mar 26, 4:59 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