Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BABEL: Added support for ISJSON(), JSON_VALUE(), and JSON_QUERY() #25

Merged
merged 2 commits into from
Nov 23, 2022

Conversation

rishabhtanwar29
Copy link

This commit introduces support for three TSQL functions: ISJSON(), JSON_VALUE(), and JSON_QUERY().

This commit uses the sql_dialect hook to toggle logic in a few common json string processing functions, in addition to adding a binary search algorithm which selects matching json keys in the event of duplicate keys at the same level.

Task: BABEL-934/935/936
Author: Brent Sienko [email protected]
Signed-off-by: Sharu Goel [email protected]
(cherry picked from commit fad820d)

thephantomthief and others added 2 commits November 23, 2022 03:46
This commit introduces support for three TSQL functions: ISJSON(),
JSON_VALUE(), and JSON_QUERY().

This commit uses the sql_dialect hook to toggle logic in a few common
json string processing functions, in addition to adding a binary search
algorithm which selects matching json keys in the event of duplicate
keys at the same level.

Task: BABEL-934/935/936
Author: Brent Sienko <[email protected]>
Signed-off-by: Sharu Goel <[email protected]>
(cherry picked from commit fad820d)
Signed-off-by: Rishabh Tanwar <[email protected]>
@rishabhtanwar29 rishabhtanwar29 merged commit 30a55d2 into babel-main Nov 23, 2022
@rishabhtanwar29 rishabhtanwar29 deleted the cherry-pick-fad820dc branch November 23, 2022 03:57
xhfanhe pushed a commit that referenced this pull request Aug 11, 2023
… (#25)

Previously, SELECT TOP (NULL) returned all rows. This commit makes it throw an error instead by adding a NULL-check in the execution of LIMIT node, and also in the planner because the planner by default skips building a LIMIT node if the LIMIT count is NULL.

Task: BABEL-1181
Author: Zitao Quan <[email protected]>
Signed-off-by: Sharu Goel <[email protected]>

Co-authored-by: Sharu Goel <[email protected]>

Co-authored-by: jusjosj <[email protected]>
Co-authored-by: Sharu Goel <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants