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

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

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

    https://github.com/apache/flink/pull/5988#discussion_r187577471
  
    --- Diff: 
flink-libraries/flink-table/src/test/scala/org/apache/flink/table/runtime/stream/sql/SqlITCase.scala
 ---
    @@ -686,6 +686,28 @@ class SqlITCase extends StreamingWithStateTestBase {
     
         assertEquals(List(expected.toString()), 
StreamITCase.testResults.sorted)
       }
    +
    +  @Test
    +  def testNullableFunctionCall(): Unit = {
    --- End diff --
    
    We should test this more lightweight with a unit test instead of running a 
complete query.
    We can extend `ScalarFunctionsTest.testLPad()` with 
    
    ```
    testSqlApi(
       "LPAD('hello', -1, 'x') IS NULL", "true"
    )
    ```
    
    to cover this case 


> Fix Codegen error of CallGenerator  
> ------------------------------------
>
>                 Key: FLINK-9332
>                 URL: https://issues.apache.org/jira/browse/FLINK-9332
>             Project: Flink
>          Issue Type: Bug
>          Components: Table API & SQL
>            Reporter: Ruidong Li
>            Assignee: Ruidong Li
>            Priority: Major
>
> function call may return null, but nullTerm did not handle it correctly.



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

Reply via email to