Page MenuHomePhabricator

Remove partial objects from Lisk
ClosedPublic

Authored by epriestley on Jul 11 2014, 1:20 PM.
Tags
None
Referenced Files
F13106891: D9895.id23944.diff
Sat, Apr 27, 9:20 PM
Unknown Object (File)
Wed, Apr 24, 10:29 PM
Unknown Object (File)
Sat, Apr 20, 7:19 PM
Unknown Object (File)
Sun, Apr 14, 4:21 AM
Unknown Object (File)
Thu, Apr 11, 9:01 AM
Unknown Object (File)
Mon, Apr 8, 10:50 AM
Unknown Object (File)
Sun, Mar 31, 4:20 AM
Unknown Object (File)
Fri, Mar 29, 3:40 AM
Subscribers

Details

Reviewers
btrahan
Maniphest Tasks
T4420: Modernize typeahead datasources
Commits
Restricted Diffusion Commit
rPc52b3c28e166: 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.

Diff Detail

Repository
rP Phabricator
Branch
tok9
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 1643
Build 1644: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

epriestley retitled this revision from to Remove partial objects from Lisk.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
btrahan edited edge metadata.
This revision is now accepted and ready to land.Jul 17 2014, 10:35 PM
epriestley updated this revision to Diff 23944.

Closed by commit rPc52b3c28e166 (authored by @epriestley).