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

Reply via email to