From 11dc2e491875fa5404dc73ba5a7ca6f44aed3456 Mon Sep 17 00:00:00 2001 From: Teddy Date: Wed, 29 Jun 2022 09:29:10 +0200 Subject: [PATCH] Fixed typo in clickhouse SQL definition (#5725) --- ingestion/src/metadata/orm_profiler/metrics/static/mean.py | 2 +- ingestion/src/metadata/orm_profiler/metrics/static/stddev.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ingestion/src/metadata/orm_profiler/metrics/static/mean.py b/ingestion/src/metadata/orm_profiler/metrics/static/mean.py index c13f507d8d1..f92791cdc88 100644 --- a/ingestion/src/metadata/orm_profiler/metrics/static/mean.py +++ b/ingestion/src/metadata/orm_profiler/metrics/static/mean.py @@ -39,7 +39,7 @@ class avg(GenericFunction): def _(element, compiler, **kw): """Handle case for empty table. If empty, clickhouse returns NaN""" proc = compiler.process(element.clauses, **kw) - return "if(isNaN(avg(%s)), null, avg(%s)" % ((proc,) * 2) + return "if(isNaN(avg(%s)), null, avg(%s))" % ((proc,) * 2) class Mean(StaticMetric): diff --git a/ingestion/src/metadata/orm_profiler/metrics/static/stddev.py b/ingestion/src/metadata/orm_profiler/metrics/static/stddev.py index 7b14b443571..c5b98d3d277 100644 --- a/ingestion/src/metadata/orm_profiler/metrics/static/stddev.py +++ b/ingestion/src/metadata/orm_profiler/metrics/static/stddev.py @@ -57,7 +57,7 @@ def _(element, compiler, **kw): If table is empty, clickhouse returns NaN. """ proc = compiler.process(element.clauses, **kw) - return "if(isNaN(stddevPop(%s)), null, 'stddevPop(%s))" % ((proc,) * 2) + return "if(isNaN(stddevPop(%s)), null, stddevPop(%s))" % ((proc,) * 2) class StdDev(StaticMetric):