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

Compute estimated size directly from Slice for parseJson function #904

Merged
merged 1 commit into from Jun 4, 2019

Conversation

3 participants
@Praveen2112
Copy link
Member

commented Jun 4, 2019

When parsing a JSON using Presto's parseJson we compute estimated size of output by populating byte[] from Slice which is not required as we could fetch the same from the length of the Slice.

Compute estimated size directly from slice
When parsing a JSON using Presto's parseJson we compute estimated size
of output by populating byte[] from Slice which is not required as we
could fetch the same from the length of the Slice

@cla-bot cla-bot bot added the cla-signed label Jun 4, 2019

@dain

dain approved these changes Jun 4, 2019

Copy link
Member

left a comment

Good catch

@dain dain merged commit ca9611c into prestosql:master Jun 4, 2019

2 checks passed

Travis CI - Pull Request Build Passed
Details
verification/cla-signed
Details

@findepi findepi added this to the 314 milestone Jun 6, 2019

@dain dain referenced this pull request Jun 7, 2019

Closed

Release notes for 314 #879

2 of 6 tasks complete
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.