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

Julian Hyde commented on CALCITE-7186:
--------------------------------------

The unit test is so narrow that I don't think it serves any purpose. I would 
rather see a UdfTest to see whether the mapping works as intended.

Related: This isn't a bug or a feature. It's just an observation that two lines 
of code are not present. Re-frame it as a bug and it's easier to test.

> Add mapping from Byte[] to VARBINARY and Character[] to VARCHAR in Java UDF
> ---------------------------------------------------------------------------
>
>                 Key: CALCITE-7186
>                 URL: https://issues.apache.org/jira/browse/CALCITE-7186
>             Project: Calcite
>          Issue Type: Improvement
>          Components: core
>    Affects Versions: 1.40.0
>            Reporter: Yu Xu
>            Assignee: Yu Xu
>            Priority: Minor
>              Labels: pull-request-available
>             Fix For: 1.41.0
>
>
> Java user-defined function (UDF) UDF functions with *Character[]* and 
> *Byte[]* return types are recognized as {*}OTHER{*}, but UDF functions with 
> *char[]* and *byte[]* return types are recognized as *VARCHAR* and 
> {*}VARBINARY{*}. *Character[]* should be consistent with {*}char[]{*}, and 
> *Byte[]* should be consistent with {*}byte[]{*}.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to