[GitHub] flink pull request #6027: [FLINK-7814][TableAPI && SQL] Add BETWEEN and NOT ...

[xccui]

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

    https://github.com/apache/flink/pull/6027#discussion_r188895005
  
    --- Diff: 
flink-libraries/flink-table/src/test/scala/org/apache/flink/table/expressions/ScalarOperatorsTest.scala
 ---
    @@ -406,5 +406,26 @@ class ScalarOperatorsTest extends 
ScalarOperatorsTestBase {
           "((((true) === true) || false).cast(STRING) + 'X ').trim",
           "trueX")
         testTableApi(12.isNull, "12.isNull", "false")
    +
    +    // between
    +    testTableApi(2.between(1, 3), "2.BETWEEN(1, 3)", "true")
    --- End diff --
    
    Would be better to add some tests for boundaries here.


---