From 7357089e4e68301a187fec50667fd9a67b125f3d Mon Sep 17 00:00:00 2001
From: weizuo93 <weizuo@apache.org>
Date: Thu, 30 Dec 2021 10:24:35 +0800
Subject: [PATCH] [fix] change percentile_approx return from nan to null
 (#7512)

Change function percentile_approx return value from nan to null (like hive.) to ensure that return value of function percentile_approxcan be parsed by JDBC successfully.
Co-authored-by: weizuo <weizuo@xiaomi.com>
---
 be/src/exprs/aggregate_functions.cpp | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/be/src/exprs/aggregate_functions.cpp b/be/src/exprs/aggregate_functions.cpp
index 231a0fdc66..1f8db84085 100644
--- a/be/src/exprs/aggregate_functions.cpp
+++ b/be/src/exprs/aggregate_functions.cpp
@@ -347,7 +347,11 @@ DoubleVal AggregateFunctions::percentile_approx_finalize(FunctionContext* ctx,
     double result = percentile->digest->quantile(quantile);
 
     delete percentile;
-    return DoubleVal(result);
+    if (isnan(result)) {
+        return DoubleVal(result).null();
+    } else {
+        return DoubleVal(result);
+    }
 }
 
 struct AvgState {