-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Add json_extract Presto function #5269
Conversation
✅ Deploy Preview for meta-velox canceled.
|
@mbasmanova has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@mbasmanova has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
@mbasmanova has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator. |
Summary: Add limited version of json_extract Presto function with support for a subset of JSONPath syntax. Pull Request resolved: facebookincubator#5269 Reviewed By: zacw7 Differential Revision: D46736261 Pulled By: mbasmanova fbshipit-source-id: 16f2ec6f04df98d9439a15c06b4836f06c2f2ed2
This pull request was exported from Phabricator. Differential Revision: D46736261 |
…ator#5267) Summary: Presto allows both JSON and VARCHAR values as first argument of json_extract_scalar. Pull Request resolved: facebookincubator#5267 Differential Revision: D46730166 Pulled By: mbasmanova fbshipit-source-id: ae822dcfb9b45fedc859fe06deb858e4c2ec663b
Summary: Add limited version of json_extract Presto function with support for a subset of JSONPath syntax. Pull Request resolved: facebookincubator#5269 Reviewed By: zacw7 Differential Revision: D46736261 Pulled By: mbasmanova fbshipit-source-id: c8758aeddc6f7e3997aa503279cbe40620f5925c
This pull request was exported from Phabricator. Differential Revision: D46736261 |
@mbasmanova merged this pull request in 3e6b0dc. |
Conbench analyzed the 1 benchmark run on commit There were no benchmark performance regressions. 🎉 The full Conbench report has more details. |
Add limited version of json_extract Presto function with support for a subset of JSONPath syntax.