Add "fields" argument to all DB API "get" methods

Bug #1771518 reported by Renat Akhmerov
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Mistral
Fix Released
Medium
Renat Akhmerov

Bug Description

We added the "fields" argument to db_api.get_workflow_execution() method so that we could load an object partially (i.e. loading the entire object may be expensive). We need to do the same for other "get" methods.

Changed in mistral:
importance: Undecided → Medium
status: New → Confirmed
tags: added: low-hanging-fruit
Changed in mistral:
milestone: none → rocky-2
Changed in mistral:
assignee: nobody → Renat Akhmerov (rakhmerov)
Changed in mistral:
status: Confirmed → In Progress
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix merged to mistral (master)

Reviewed: https://review.openstack.org/569049
Committed: https://git.openstack.org/cgit/openstack/mistral/commit/?id=eadce20f479b1f385dc2ab0f54858a40208cb408
Submitter: Zuul
Branch: master

commit eadce20f479b1f385dc2ab0f54858a40208cb408
Author: Renat Akhmerov <email address hidden>
Date: Thu May 17 14:57:02 2018 +0700

    Add "fields" argument to DB API methods

    * Added the "fields" argument to all "get" and "load" DB API
      methods where we potentially need partial loading
    * Removed an unused method
    * Refactored some methods taking "namespace" as an argument

    Closes-Bug: #1771518
    Change-Id: I3eb094c24bc5309d05bf859c0b4d43487d50f1df

Changed in mistral:
status: In Progress → Fix Released
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix included in openstack/mistral 7.0.0.0b2

This issue was fixed in the openstack/mistral 7.0.0.0b2 development milestone.

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.