Airblader commented on a change in pull request #17275:
URL: https://github.com/apache/flink/pull/17275#discussion_r710918324
##########
File path: docs/data/sql_functions.yml
##########
@@ -652,19 +652,23 @@ json:
behavior for `ON EMPTY`, and raises an error for `ON ERROR`.
```
- // STRING: "true"
+ -- "true"
JSON_VALUE('{"a": true}', '$.a')
- // BOOLEAN: true
+ -- TRUE
JSON_VALUE('{"a": true}', '$.a' RETURNING BOOLEAN)
- // STRING: "false"
+ -- "false"
JSON_VALUE('{"a": true}', 'lax $.b'
DEFAULT FALSE ON EMPTY)
- // STRING: "false"
+ -- "false"
JSON_VALUE('{"a": true}', 'strict $.b'
DEFAULT FALSE ON ERROR)
+
+ -- 0.998D
+ JSON_VALUE('{"a.b": [0.998,0.996]}','$.[''a.b''][0]'
Review comment:
I think even SQL-savy people would expect this here, because this is a
JSON function, and in JSON you use double quotes. So double quotes actually
kind of make more sense to me than trying to use single quotes.
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
