Uh oh!
There was an error while loading. Please reload this page.
PHOENIX-1661 Implement built-in functions for JSON - #102
Closed
wlei-llvm wants to merge 27 commits into
Closed
Conversation
add a function to support "ArrayToJson" function as a build-in function for JSON
a simple test for testing ArrayToJsonFunction
Implement built-in functions for JSON
PHOENIX-1661 Implement built-in functions for JSON
Conflicts: phoenix-core/src/main/java/org/apache/phoenix/schema/json/PhoenixJson.java
2 add some javadocs and commemts in the code 3 define variables only at the place where it is used 4 use StringBuilder replace "+" operator 5 rename some method name in PhoenixJson 6 change some helper methods level in JsonFunctionTest
merge master branch
add SQLExceptionCode fix bugs
Contributor
There was a problem hiding this comment.
This (and other occurences of the same) should return true, meaning "yes, there was enough information available for me to evaluate this expression, but it returned null (which is what a ptr.getLength() of 0 means)"
Contributor
There was a problem hiding this comment.
Rename this to json_each_text (do not implement json_each)
This was referenced Dec 30, 2020
This was referenced Jan 6, 2021
This was referenced Feb 19, 2021
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add some built-in functions for JSON : JsonArrayElementsFunction.java JsonArrayLengthFunction.java JsonEachFunction.java JsonObjectKeysFunction.java JsonPopulateRecordFunction.java JsonPopulateRecordSetFunction.java. ArrayToJsonFunction.java
Add end2end test files.
Modify file PhoenixJson.java to which I added some helper methods.
@JamesRTaylor@twdsilva@AakashPradeep