[jira] [Commented] (FLINK-9970) Add ASCII/CHR function for table/sql API

["ASF GitHub Bot (JIRA)" <ji...@apache.org>]

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

ASF GitHub Bot commented on FLINK-9970:
---------------------------------------

yanghua commented on a change in pull request #6432: [FLINK-9970] [table] Add ASCII/CHR function for table/sql API
URL: https://github.com/apache/flink/pull/6432#discussion_r206741741
 
 

 ##########
 File path: flink-libraries/flink-table/src/main/scala/org/apache/flink/table/codegen/calls/FunctionGenerator.scala
 ##########
 @@ -146,12 +146,25 @@ object FunctionGenerator {
     STRING_TYPE_INFO,
     BuiltInMethod.OVERLAY.method)
 
+  addSqlFunctionMethod(
+    ASCII,
+    Seq(STRING_TYPE_INFO),
+    STRING_TYPE_INFO,
 
 Review comment:
   @suez1224 thanks for your reminder. You are right, In the scenario you mentioned, it throws an exception. I found that we don't need to check if the input of the function is null. It seems that the Flink framework has been processed by default. All the input is null and will return null (man can not interfere). Then we only need to deal with "", in this case, the ASCII code is 0. So, I directly changed the return value of this method to the Int type. 
   
   cc @hequn8128 : 
   
   It seems that we don't need to consider the case where the input is null. Once the input of the function is null, this function will not be called at all. I debugged it through the test class. It looks like this, you can confirm it.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


> Add ASCII/CHR function for table/sql API
> ----------------------------------------
>
>                 Key: FLINK-9970
>                 URL: https://issues.apache.org/jira/browse/FLINK-9970
>             Project: Flink
>          Issue Type: Sub-task
>          Components: Table API & SQL
>            Reporter: vinoyang
>            Assignee: vinoyang
>            Priority: Minor
>              Labels: pull-request-available
>
> for ASCII function : 
> refer to : [https://dev.mysql.com/doc/refman/8.0/en/string-functions.html#function_ascii]
> for CHR function : 
> This function convert ASCII code to a character,
> refer to : [https://doc.ispirer.com/sqlways/Output/SQLWays-1-071.html]
> Considering "CHAR" always is a keyword in many database, so we use "CHR" keyword.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)