Should add a way to get the entire object for deletion (local_controller)

Bug #1646079 reported by Shachar Snapiri
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
DragonFlow
New
Low
Unassigned

Bug Description

All the object delete methods in the local controller (e.g. logical_switch_deleted) get the object ID, only to fetch it from the DB store. In most cases the code calling these methods has the object at hand, and sends the object.get_id().

There are several aspects to it:
1. We have the object at hand, but send its ID only to re-fetch it (performance issue)
2. The fetching is from the db_store, which is in-memory, so the cost is not too big (but nevertheless)
3. There are places that call these methods and only have the ID at hand, so we would have to have a way to keep the current functionality for these places in the code
Every item here has its advantages and disadvantages, a decision should be made.

Changed in dragonflow:
assignee: nobody → Shachar Snapiri (snapiri)
Li Ma (nick-ma-z)
Changed in dragonflow:
importance: Undecided → Low
Changed in dragonflow:
assignee: Shachar Snapiri (snapiri) → nobody
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.