From cd347299d75eb10feda8914e7e24470612da90aa Mon Sep 17 00:00:00 2001 From: mgorsk1 Date: Thu, 20 Jul 2023 16:50:24 +0200 Subject: [PATCH] Fixes 12439: Remove braces from approx_percentile method (Trino/Presto) in profiler (#12440) * :bug: Remove braces from approx_percentile method (Trino/Presto) * :ok_hand: Updating code due to code review changes. --- ingestion/src/metadata/profiler/orm/functions/median.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/ingestion/src/metadata/profiler/orm/functions/median.py b/ingestion/src/metadata/profiler/orm/functions/median.py index e0b3106d927..dbd36fbe2e4 100644 --- a/ingestion/src/metadata/profiler/orm/functions/median.py +++ b/ingestion/src/metadata/profiler/orm/functions/median.py @@ -58,7 +58,7 @@ def _(elements, compiler, **kwargs): def _(elements, compiler, **kwargs): col = compiler.process(elements.clauses.clauses[0]) percentile = elements.clauses.clauses[2].value - return 'approx_percentile("%s", %.2f)' % (col, percentile) + return "approx_percentile(%s, %.2f)" % (col, percentile) @compiles(MedianFn, Dialects.MSSQL)