Homec4science

Remove ORDER_PATH_MODIFIED from Differential

Authored by epriestley <git@epriestley.com> on Apr 12 2015, 05:16.

Description

Remove ORDER_PATH_MODIFIED from Differential

Summary:
Ref T7803. This is a performance hack, not a real order, and isn't really meaningful or pageable.

After D12158, we constraint his query on dateModified anyway, which should generally give the database a relatively small result set to examine.

Test Plan: Browsed Differential and Diffusion. Checked query plan, it didn't look too crazy.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T7803

Differential Revision: https://secure.phabricator.com/D12361

Details

Committed
epriestley <git@epriestley.com>Apr 13 2015, 20:58
Pushed
aubortJan 31 2017, 17:16
Parents
rPHe0fa0fbdee1b: Modernize Phriction ordering/paging
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPH9b5198f46330: Remove ORDER_PATH_MODIFIED from Differential (authored by epriestley <git@epriestley.com>).Apr 13 2015, 20:58