Homec4science

Remove partial objects from Lisk

Authored by epriestley <git@epriestley.com> on Jul 18 2014, 00:49.

Description

Remove partial objects from Lisk

Summary:
Ref T4420. This was a performance hack introduced long ago to make typeaheads for users a little cheaper. The idea was that you could load some of an object's columns and skip other ones.

We now always load users on demand, so the cost of loading the whole objects is very small. No other use cases ever arose for this, and it seems unlikely that they will in the future. Remove it all.

Test Plan:

  • Grepped for CONFIG_PARTIAL_OBJECTS.
  • Grepped for dirtyFields.
  • Grepped for missingFields.
  • Grepped for resetDirtyFields.
  • Grepped for loadColumns.
  • Grepped for loadColumnsWhere.
  • Grepped for loadRawDataWhere.
  • Loaded and saved some lisk objects.

Reviewers: btrahan

Reviewed By: btrahan

Subscribers: epriestley

Maniphest Tasks: T4420

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

Details

Committed
epriestley <git@epriestley.com>Jul 18 2014, 00:49
Pushed
aubortJan 31 2017, 17:16
Parents
rPHb8d604acaf84: Make typeahead datasources default to PHID type icons
Branches
Unknown
Tags
Unknown

Event Timeline

epriestley <git@epriestley.com> committed rPHc52b3c28e166: Remove partial objects from Lisk (authored by epriestley <git@epriestley.com>).Jul 18 2014, 00:49