davidradl commented on code in PR #26022:
URL: https://github.com/apache/flink/pull/26022#discussion_r1928632545


##########
docs/data/sql_functions.yml:
##########
@@ -1069,6 +1072,30 @@ json:
         )
       )
       ```
+  - sql: JSON(value)

Review Comment:
   I understand there is a new json type - please could you point me to 
databases that have a SQL function called JSON().
   
   I look at 
[Oracle](https://docs.oracle.com/en/database/oracle/oracle-database/12.2/adjsn/generation.html#GUID-1084A518-A44A-4654-A796-C1DD4D8EC2AA)
 and see that they use json_object for the nested json_object.
   also I see there is an example of a nested object using json_object in the 
[flink 
docs](https://nightlies.apache.org/flink/flink-docs-release-1.20/docs/dev/table/functions/systemfunctions/):
   
   ```
   JSON_OBJECT(
     KEY 'K1'
     VALUE JSON_OBJECT(
       KEY 'K2'
       VALUE 'V'
     )
   )
   ```
   
   why do we need a new function when we can already do this with json_object? 
Or am I missing something?
   



-- 
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.

To unsubscribe, e-mail: issues-unsubscr...@flink.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to