Join multiple criteria together

Bug #1449850 reported by Dave Chen
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Glance
In Progress
Low
Kamil Rykowski
OpenStack Identity (keystone)
Won't Fix
Low
Dave Chen

Bug Description

SQLAlchemy supports to join multiple criteria together, this is provided to build the query statements when there is multiple filtering criterion instead of constructing query statement one by one, just *assume* SQLAlchemy prefer to use it in this way, and the code looks more clean after refactoring.

Dave Chen (wei-d-chen)
Changed in keystone:
assignee: nobody → Dave Chen (wei-d-chen)
Dave Chen (wei-d-chen)
description: updated
Changed in keystone:
status: New → In Progress
Revision history for this message
Lin Hua Cheng (lin-hua-cheng) wrote :
Changed in keystone:
milestone: none → liberty-1
Brant Knudson (blk-u)
Changed in keystone:
status: In Progress → Opinion
Changed in keystone:
status: Opinion → In Progress
Dolph Mathews (dolph)
Changed in keystone:
importance: Undecided → Low
Changed in glance:
status: New → In Progress
assignee: nobody → Kamil Rykowski (kamil-rykowski)
importance: Undecided → Low
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Fix proposed to glance (master)

Fix proposed to branch: master
Review: https://review.openstack.org/182631

Revision history for this message
Dave Chen (wei-d-chen) wrote :

As the comments in this patch: (https://review.openstack.org/#/c/133135/), we won't fix this this time.

Changed in keystone:
status: In Progress → Won't Fix
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on keystone (master)

Change abandoned by Dave Chen (<email address hidden>) on branch: master
Review: https://review.openstack.org/133135
Reason: As the comments on this patch, we won't fix it this time.

Changed in keystone:
milestone: liberty-1 → none
Revision history for this message
OpenStack Infra (hudson-openstack) wrote : Change abandoned on glance (master)

Change abandoned by Glance Bot (<email address hidden>) on branch: master
Review: https://review.openstack.org/182631

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.