From ce6fb1cfba88658469f4487cf9dd043caa451e86 Mon Sep 17 00:00:00 2001
From: EmmyMiao87 <522274284@qq.com>
Date: Wed, 25 Sep 2019 22:00:02 +0800
Subject: [PATCH] Fix bug: broker load not support inline function in hll_hash
 (#1873)

hll_hash should support the inline function in broker load and should not support the inline function in hadoop load.
---
 .../java/org/apache/doris/analysis/DataDescription.java    | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/fe/src/main/java/org/apache/doris/analysis/DataDescription.java b/fe/src/main/java/org/apache/doris/analysis/DataDescription.java
index e3fce84ef2..40c47cb885 100644
--- a/fe/src/main/java/org/apache/doris/analysis/DataDescription.java
+++ b/fe/src/main/java/org/apache/doris/analysis/DataDescription.java
@@ -67,6 +67,7 @@ import java.util.TreeSet;
 public class DataDescription {
     private static final Logger LOG = LogManager.getLogger(DataDescription.class);
     public static String FUNCTION_HASH_HLL = "hll_hash";
+    // function isn't built-in function, hll_hash is not built-in function in hadoop load.
     private static final List<String> HADOOP_SUPPORT_FUNCTION_NAMES = Arrays.asList(
             "strftime",
             "time_format",
@@ -333,8 +334,10 @@ public class DataDescription {
             } else if (paramExpr instanceof NullLiteral) {
                 args.add(null);
             } else {
-                // hadoop function only support slot, string and null parameters
-                throw new AnalysisException("Mapping function args error, arg: " + paramExpr.toSql());
+                if (isHadoopLoad) {
+                    // hadoop function only support slot, string and null parameters
+                    throw new AnalysisException("Mapping function args error, arg: " + paramExpr.toSql());
+                }
             }
         }