Page MenuHomePhabricator

Remove partial objects from Lisk
ClosedPublic

Authored by epriestley on Jul 11 2014, 1:20 PM.
Tags
None
Referenced Files
F13291745: D9895.diff
Wed, Jun 5, 12:28 AM
F13290563: D9895.diff
Tue, Jun 4, 6:39 PM
F13286669: D9895.diff
Tue, Jun 4, 7:33 AM
F13272080: D9895.diff
Thu, May 30, 7:44 AM
F13269468: D9895.diff
Wed, May 29, 6:21 AM
F13258582: D9895.id23755.diff
Sun, May 26, 10:48 AM
F13253216: D9895.diff
Sat, May 25, 2:30 AM
F13246188: D9895.id23755.diff
Thu, May 23, 8:29 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).