Page MenuHomePhabricator

Remove recently added repository_pathchange key
ClosedPublic

Authored by epriestley on Mar 16 2016, 9:50 PM.
Tags
None
Referenced Files
F18839387: D15486.diff
Mon, Oct 27, 4:02 PM
F18829803: D15486.id37321.diff
Sat, Oct 25, 1:14 AM
F18783148: D15486.id37322.diff
Mon, Oct 13, 4:12 AM
F18777281: D15486.id.diff
Sat, Oct 11, 1:44 AM
F18701572: D15486.id.diff
Sep 27 2025, 7:58 PM
F18690906: D15486.diff
Sep 26 2025, 8:19 PM
F18643058: D15486.diff
Sep 19 2025, 1:20 AM
F18622106: D15486.id.diff
Sep 15 2025, 10:30 AM
Subscribers
None

Details

Summary

Ref T10560. Reverts D15460. See that task for discussion: we dug up some more information to explain the behavior, and this key was just sort of sidestepping an analyze/cardinality estimate issue on the index.

With proper cardinality estimates it shouldn't be used, so just nuke it.

Test Plan

Ran bin/storage adjust, saw key drop.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Remove recently added repository_pathchange key.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, eadler.
chad edited edge metadata.
This revision is now accepted and ready to land.Mar 16 2016, 9:52 PM
This revision was automatically updated to reflect the committed changes.