From fc0361a897b4683740b75feca82c022b0b1465e0 Mon Sep 17 00:00:00 2001 From: Zane Bitter Date: Mon, 2 Sep 2013 16:02:14 +0200 Subject: [PATCH] Don't use a query for watch_rule_get() When getting a database record by ID, it is much more efficient to get() it directly than to perform a query or the database to obtain the ID (again) and then get it. Change-Id: I89b46d698c033e54ee9a50172ccad3eb3833cb1e --- heat/db/sqlalchemy/api.py | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/heat/db/sqlalchemy/api.py b/heat/db/sqlalchemy/api.py index dd1716f2..4c1ac228 100644 --- a/heat/db/sqlalchemy/api.py +++ b/heat/db/sqlalchemy/api.py @@ -315,8 +315,7 @@ def event_create(context, values): def watch_rule_get(context, watch_rule_id): - result = model_query(context, models.WatchRule).\ - filter_by(id=watch_rule_id).first() + result = model_query(context, models.WatchRule).get(watch_rule_id) return result -- 2.45.2