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

ASF GitHub Bot commented on DRILL-7867:
---------------------------------------

vvysotskyi commented on pull request #2179:
URL: https://github.com/apache/drill/pull/2179#issuecomment-783970874


   Hi @luocooong, no need to add a new unit test, since this functionality is 
already checked by the `TestStringFunctions.testSplit` unit test. We have only 
split UDF that accepts nullable inputs (UDF that accepts required arguments was 
removed in this PR).


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
[email protected]


> Allow using required arguments for functions with nullable parameters
> ---------------------------------------------------------------------
>
>                 Key: DRILL-7867
>                 URL: https://issues.apache.org/jira/browse/DRILL-7867
>             Project: Apache Drill
>          Issue Type: Improvement
>    Affects Versions: 1.19.0
>            Reporter: Vova Vysotskyi
>            Assignee: Vova Vysotskyi
>            Priority: Major
>             Fix For: 1.19.0
>
>
> When the user introduces only a single UDF that accepts nullable arguments, 
> it should be possible to use that UDF for arguments with the required data 
> mode.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to