openstack/deb-ceilometer
Code Issues Proposed changes

mysql doesn't understand intersect

Browse Source 
get_events currently uses intersect to build results. mysql doesn't
necessarily understand intersect so we change the query to be
equivalent query using exists filter

Change-Id: I0960010a503b8a90b62f150831a1d138ecd396bc
Closes-Bug: #1423713
This commit is contained in:
gordon chung 2015-02-19 17:54:58 -05:00
parent 534048d96c
commit b32e28d8c0
1 changed files with 13 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
ceilometer/event/storage/impl_sqlalchemy.py
View File

@ -70,7 +70,7 @@ def _build_trait_query(session, trait_type, key, value, op='eq'):
'ne': (trait_model.value != value)}
conditions = [trait_model.key == key, op_dict[op]]
return (session.query(trait_model.event_id.label('ev_id'))
.filter(*conditions))
.filter(*conditions), trait_model)
class Connection(base.Connection):
@ -254,16 +254,21 @@ class Connection(base.Connection):
trait_subq = None
# Build trait filter
if event_filter.traits_filter:
trait_qlist = []
for trait_filter in event_filter.traits_filter:
filters = list(event_filter.traits_filter)
trait_filter = filters.pop()
key = trait_filter.pop('key')
op = trait_filter.pop('op', 'eq')
trait_qlist.append(_build_trait_query(
trait_subq, t_model = _build_trait_query(
session, trait_filter.keys()[0], key,
trait_filter.values()[0], op))
trait_subq = trait_qlist.pop()
if trait_qlist:
trait_subq = trait_subq.intersect(*trait_qlist)
trait_filter.values()[0], op)
for trait_filter in filters:
key = trait_filter.pop('key')
op = trait_filter.pop('op', 'eq')
q, model = _build_trait_query(
session, trait_filter.keys()[0], key,
trait_filter.values()[0], op)
trait_subq = trait_subq.filter(
q.filter(model.event_id == t_model.event_id).exists())
trait_subq = trait_subq.subquery()
query = (session.query(models.Event.id)