1
0
Fork 0
mirror of https://we.phorge.it/source/phorge.git synced 2024-11-27 09:12:41 +01:00

Remove recently added repository_pathchange key

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.

Reviewers: eadler, chad

Reviewed By: chad

Maniphest Tasks: T10560

Differential Revision: https://secure.phabricator.com/D15486
This commit is contained in:
epriestley 2016-03-16 14:48:51 -07:00
parent 772c658aac
commit 5520729db3

View file

@ -155,9 +155,6 @@ final class PhabricatorRepositorySchemaSpec
'repositoryID' => array(
'columns' => array('repositoryID', 'pathID', 'commitSequence'),
),
'key_history' => array(
'columns' => array('commitID', 'isDirect', 'changeType'),
),
));
$this->buildRawSchema(