MDEV-13594: Implement -> and ->> JSON path operators#4899
MDEV-13594: Implement -> and ->> JSON path operators#4899kjarir wants to merge 1 commit intoMariaDB:mainfrom
Conversation
|
@gkodinov @grooverdan There was a huge mistake from my side, so i moved the PR to this. |
gkodinov
left a comment
There was a problem hiding this comment.
Since this is a new pull request on the same change, please reply to all the questions in the closed pull request.
| %expect 72 | ||
| %else | ||
| %expect 71 | ||
| %expect 73 |
There was a problem hiding this comment.
In the interest of more consistent processing, can you please answer all questions from the previous pull request? E.g. I had a question on this line. And I see that it's still unaddressed.
|
Also, if you need help with github and pull requests management, please reach out to us on Zulip. |
|
@gkodinov , I sincerely apologize for the mismanagement. My previous PR branch was accidentally deleted due to a mistake on my end, which forced me to open this new PR. I should have reached out for help on Zulip instead of handling it this way. I will make sure this does not happen again. I will also be responding to every review question from the previously closed PR (#4711) shortly. |
gkodinov
left a comment
There was a problem hiding this comment.
LGTM. Please stand by for the final review.
[MDEV-13594] Implement JSON
->and->>OperatorsSummary
This PR implements MySQL-compatible JSON shorthand operators in the MariaDB server. These operators provide a more concise syntax for extracting and unquoting JSON data:
column->'$.path'is equivalent toJSON_EXTRACT(column, '$.path')column->>'$.path'is equivalent toJSON_UNQUOTE(JSON_EXTRACT(column, '$.path'))Details
-character state to identify->and->>sequences insql/sql_lex.cc.sql/sql_yacc.yyunderprimary_expr. To resolve shift/reduce ambiguities, the Left-Hand Side (LHS) has been restricted toprimary_base_expr(non-recursive).Performance & Quality
json.mdev13594.Status: [ pass ]