libenchao commented on a change in pull request #11186: [FLINK-16200][sql] Support JSON_EXISTS for blink planner URL: https://github.com/apache/flink/pull/11186#discussion_r396857138
########## File path: flink-table/flink-table-planner-blink/src/test/scala/org/apache/flink/table/planner/expressions/ScalarFunctionsTest.scala ########## @@ -4195,4 +4195,48 @@ class ScalarFunctionsTest extends ScalarTypesTestBase { "f55=f57", "true") } + + //------------------------------------------------------------------- + // JSON functions + //------------------------------------------------------------------- + @Test + def testJsonExists(): Unit = { Review comment: Do you mean supporting these json functions in SQL first? Or just testing for SQL? If the answer is supporting these json functions in SQL firstly, I prefer we do not add these table api related expressions for now, which are not tested for now. ---------------------------------------------------------------- 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: us...@infra.apache.org With regards, Apache Git Services