Merge ijson (#563) to 2.0 #576
Merged
Conversation
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
* initial ijson commit * Code arrangement * Return serde_json:Value support * Implement manager for IValue * Implement lasts unimplimented manager API for IValue * fmt fixes * return jsonpath library to point to generic_json_path * Made backend configurable on start using JSON_BACKEND module argument that can get either SERDE_JSON or IJSON, default is IJSON. * run tests on both backends * json_init -> json_init_config * fmt fixes * tests fixes * fmt fixes * disable memory test as its currently incorrect * review fixes * update redismodule-rs to 0.25 * fix 6.0 tests * Skip short read test on 6.0 (too slow) * Skip module keyspace notification test (introduced in redis 6.2) Co-authored-by: meir <meir@redis.com> Co-authored-by: oshadmi <omer.shadmi@redislabs.com> (cherry picked from commit b6a070d)
rafie
previously approved these changes
Dec 14, 2021
(cherry picked from commit 57e6a87)
(cherry picked from commit 8cf0cd4)
Co-authored-by: Rafi Einstein <rafi@redislabs.com> (cherry picked from commit 42e6f32)
* Update freebsd.yml (cherry picked from commit 58d6fbf)
* JSON.SET full doc benchmark extensions * [fix] Adjusted json_vs_hashes_json.set_key_simple to use a larger keyspace range * [fix] Fixed q5 JSON.SET test (cherry picked from commit 3ec45da)
Bumps [mkdocs](https://github.com/mkdocs/mkdocs) from 1.1.2 to 1.2.3. - [Release notes](https://github.com/mkdocs/mkdocs/releases) - [Commits](mkdocs/mkdocs@1.1.2...1.2.3) --- updated-dependencies: - dependency-name: mkdocs dependency-type: direct:production ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> (cherry picked from commit 91892dd)
|
|
rafie
approved these changes
Dec 15, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
initial ijson commit
Code arrangement
Return serde_json:Value support
Implement manager for IValue
Implement lasts unimplimented manager API for IValue
fmt fixes
return jsonpath library to point to generic_json_path
Made backend configurable on start using JSON_BACKEND module argument that can get either SERDE_JSON or IJSON, default is IJSON.
run tests on both backends
json_init -> json_init_config
fmt fixes
tests fixes
fmt fixes
disable memory test as its currently incorrect
review fixes
update redismodule-rs to 0.25
fix 6.0 tests
Skip short read test on 6.0 (too slow)
Skip module keyspace notification test (introduced in redis 6.2)
Co-authored-by: meir meir@redis.com
Co-authored-by: oshadmi omer.shadmi@redislabs.com
(cherry picked from commit b6a070d)