ritesh-kapoor commented on a change in pull request #1107: [CALCITE-2870]
Support for JSON query quotes behavior
URL: https://github.com/apache/calcite/pull/1107#discussion_r352666656
##########
File path:
core/src/test/java/org/apache/calcite/sql/test/SqlOperatorBaseTest.java
##########
@@ -4465,6 +4465,14 @@ private void checkNullOperand(SqlTester tester, String
op) {
+ "with conditional wrapper)",
"[100]", "VARCHAR(2000)");
+ //omit quotes test
+ tester.checkString("json_query('{\"foo\":\"100\"}', 'strict $.foo' "
Review comment:
I can raise a separate PR for the optional clause for JSON_QUERY. Also
1. If OMIT QUOTES is specified, return a string value;
2. If KEEP QUOTES is specified, the function would behave as if it was
unspecified.
3. If quote behavior is not specified, the function would behave as if it
was unspecified.
Got some references :
https://marc.info/?l=postgresql-hackers&m=153024002012585&w=2
Sorry this PR is pending for long, I will also resolve all the conflicts
once it is ready to be merged.
----------------------------------------------------------------
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:
[email protected]
With regards,
Apache Git Services
