Skip to content

Commit 00cc089

Browse files
[gaarf-py] Perform jinja expansion only for exisiting template variable in the query
Change-Id: I42b5ab1d46bf2510e58971c66f07480584513992
1 parent 5c6ea71 commit 00cc089

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

py/gaarf/bq_executor.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -41,9 +41,14 @@ def execute(
4141
logger.debug("Original query text:\n%s", query_text)
4242
if params:
4343
if (templates := params.get("template")):
44-
query_text = expand_jinja(query_text, **templates)
45-
logger.debug("Query text after jinja expansion:\n%s",
46-
query_text)
44+
query_templates = {
45+
name: value
46+
for name, value in templates.items() if name in query_text
47+
}
48+
if query_templates:
49+
query_text = expand_jinja(query_text, **query_templates)
50+
logger.debug("Query text after jinja expansion:\n%s",
51+
query_text)
4752
if (macros := params.get("macro")):
4853
query_text = query_text.format(**macros)
4954
logger.debug("Query text after macro substitution:\n%s",

0 commit comments

Comments
 (0)