You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@impala.apache.org by jo...@apache.org on 2021/06/23 03:59:56 UTC

[impala] 02/03: IMPALA-10760: Added accessor methods to Function classes

This is an automated email from the ASF dual-hosted git repository.

joemcdonnell pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/impala.git

commit 078e1fddb08a84a76374592f4ae518e2ff68e56f
Author: Steve Carlin <sc...@cloudera.com>
AuthorDate: Mon Jun 21 16:57:10 2021 -0700

    IMPALA-10760: Added accessor methods to Function classes
    
    Small modification to allow the rest of the "symbol" strings
    to be visible outside the Function classes.
    
    These are necessary for an external frontend to access these
    symbols.
    
    Change-Id: I4b42c1a9cbca7f421cf8cbcf79890eb4a5ac3582
    Reviewed-on: http://gerrit.cloudera.org:8080/17614
    Reviewed-by: Aman Sinha <am...@cloudera.com>
    Tested-by: Aman Sinha <am...@cloudera.com>
---
 fe/src/main/java/org/apache/impala/catalog/AggregateFunction.java | 2 ++
 fe/src/main/java/org/apache/impala/catalog/ScalarFunction.java    | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/fe/src/main/java/org/apache/impala/catalog/AggregateFunction.java b/fe/src/main/java/org/apache/impala/catalog/AggregateFunction.java
index 6b7b686..b51c8ce 100644
--- a/fe/src/main/java/org/apache/impala/catalog/AggregateFunction.java
+++ b/fe/src/main/java/org/apache/impala/catalog/AggregateFunction.java
@@ -211,6 +211,8 @@ public class AggregateFunction extends Function {
   public String getSerializeFnSymbol() { return serializeFnSymbol_; }
   public String getMergeFnSymbol() { return mergeFnSymbol_; }
   public String getFinalizeFnSymbol() { return finalizeFnSymbol_; }
+  public String getRemoveFnSymbol() { return removeFnSymbol_; }
+  public String getValueFnSymbol() { return getValueFnSymbol_; }
   public boolean ignoresDistinct() { return ignoresDistinct_; }
   public boolean isAnalyticFn() { return isAnalyticFn_; }
   public boolean isAggregateFn() { return isAggregateFn_; }
diff --git a/fe/src/main/java/org/apache/impala/catalog/ScalarFunction.java b/fe/src/main/java/org/apache/impala/catalog/ScalarFunction.java
index 095413b..2f8d5ed 100644
--- a/fe/src/main/java/org/apache/impala/catalog/ScalarFunction.java
+++ b/fe/src/main/java/org/apache/impala/catalog/ScalarFunction.java
@@ -266,6 +266,8 @@ public class ScalarFunction extends Function {
   public void setCloseFnSymbol(String s) { closeFnSymbol_ = s; }
 
   public String getSymbolName() { return symbolName_; }
+  public String getPrepareFnSymbol() { return prepareFnSymbol_; }
+  public String getCloseFnSymbol() { return closeFnSymbol_; }
 
   @Override
   protected TSymbolLookupParams getLookupParams() {