[ 
https://issues.apache.org/jira/browse/PHOENIX-3355?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15655628#comment-15655628
 ] 

ASF GitHub Bot commented on PHOENIX-3355:
-----------------------------------------

Github user maryannxue commented on a diff in the pull request:

    https://github.com/apache/phoenix/pull/222#discussion_r87512477
  
    --- Diff: 
phoenix-core/src/main/java/org/apache/phoenix/parse/FunctionParseNode.java ---
    @@ -30,6 +30,7 @@
     import java.util.List;
    --- End diff --
    
    Probably there will be no need for any change in this file other than 
adding one more annotation attribute and one more field in BuiltinFunctionInfo. 
Let's move the "overloadArguments()" to PhoenixSchema.


> Register Phoenix built-in functions as Calcite functions
> --------------------------------------------------------
>
>                 Key: PHOENIX-3355
>                 URL: https://issues.apache.org/jira/browse/PHOENIX-3355
>             Project: Phoenix
>          Issue Type: Bug
>            Reporter: James Taylor
>            Assignee: Eric Lomore
>              Labels: calcite
>         Attachments: PHOENIX-3355.function_constructor.patch, 
> PHOENIX-3355.wip, PHOENIX-3355.wip2
>
>
> We should register all Phoenix built-in functions that don't exist in Calcite.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to