You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@calcite.apache.org by "Feng Zhu (Jira)" <ji...@apache.org> on 2020/01/03 07:54:00 UTC
[jira] [Commented] (CALCITE-3395) add BuiltinMethod for
Substring(String, int)
[ https://issues.apache.org/jira/browse/CALCITE-3395?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17007285#comment-17007285 ]
Feng Zhu commented on CALCITE-3395:
-----------------------------------
Hi, [~yuanyoujun].
In CALCITE-3563, we have added mechnism in code generation to intelligently figure out the proper method through overloaded ones.
I think it can solve this problem. Could you verify it?
> add BuiltinMethod for Substring(String, int)
> --------------------------------------------
>
> Key: CALCITE-3395
> URL: https://issues.apache.org/jira/browse/CALCITE-3395
> Project: Calcite
> Issue Type: Improvement
> Components: core
> Reporter: Youjun Yuan
> Priority: Minor
> Original Estimate: 4h
> Remaining Estimate: 4h
>
> substring function has two versoins:
> 1, Substring(String, int, int)
> 2, Substring(String, int)
> currently in BuiltinMethod.java, only the first one is defined. Need to define the second one as well, so that we can use both of them.
> Apache Flink(FunctionGenerator.scala), reference the BuiltinMethod, since Calcite only defines 1, not 2, Flink always resolve SUBSTRING to Substring(String, int, int) even if there is only 2 parameters. This problem happens to be covered by method overloading of java, but it's still a potential problem.
--
This message was sent by Atlassian Jira
(v8.3.4#803005)