Page MenuHomePhabricator

Modernize Drydock Query + Attach code
ClosedPublic

Authored by epriestley on Sep 15 2015, 5:01 PM.
Tags
None
Referenced Files
F18712745: D14113.id.diff
Mon, Sep 29, 5:31 AM
F18704895: D14113.id34104.diff
Sun, Sep 28, 9:17 AM
F18648202: D14113.id34104.diff
Sep 19 2025, 3:59 PM
F18499084: D14113.diff
Sep 4 2025, 7:23 PM
F18455829: D14113.id.diff
Sep 1 2025, 9:48 AM
F18442609: D14113.diff
Aug 31 2025, 5:06 PM
F18086146: D14113.id34104.diff
Aug 5 2025, 7:41 PM
F18085208: D14113.diff
Aug 5 2025, 7:03 PM
Subscribers
None

Details

Summary

Ref T9253. Some of the Drydock code is pretty old. This applies standard modernizations to it:

  • Modernize Query classes to use stuff like buildWhereClauseParts() and loadStandardPage().
  • Modernize all the getX() / attachX() stuff. In particular:
    • Require and attach implementations to Blueprints.
    • Require and attach Blueprints to Resources.
    • BlueprintImplementations are now always unique per-Blueprint so they can store/cache state if they want without running over one another.
    • BlueprintImplementations are now passed a $blueprint, like other similar APIs (this could go various ways but I generally like this as a balance of concerns).
NOTE: This probably doesn't run on its own, I'm just trying to split the next diff (core allocator stuff) up a bit and these pieces are all pretty standard.
Test Plan
  • Not much; see next revision or two.
  • Clicked around Resource and Blueprint lists.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Modernize Drydock Query + Attach code.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added reviewers: chad, hach-que.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 15 2015, 6:04 PM
hach-que edited edge metadata.
This revision was automatically updated to reflect the committed changes.