MOD-2785: Add benchmark for issue #674 #693
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
Codecov Report
@@ Coverage Diff @@
## master #693 +/- ##
=======================================
Coverage 83.28% 83.28%
=======================================
Files 11 11
Lines 3236 3236
=======================================
Hits 2695 2695
Misses 541 541 Continue to review full report at Codecov.
|
…escent_with_filter_uid_issue674 benchmark
filipecosta90
approved these changes
Mar 16, 2022
oshadmi
added a commit
that referenced
this pull request
Mar 26, 2022
* Merge from master to 2.0 (towards 2.0.7) (#665) * Fixes in config.yml (#409) (cherry picked from commit 8adfd29) * Updated module version * json.get returns a top-level array (as bulk string) (#411) (#412) * json.get returns a top-level array (as bulk string) * Fix format and tests (cherry picked from commit cc6d225) * Updated modules version (2) * Added integration branch 2.0 * Updated module version * Updated modules version (2) * Added integration branch 2.0 * [2.0] Updated readies * [2.0] Updated readies (for docker cpuset) * update readies to master * Update version to 2.0.2 * Update Cargo.toml version to 2.0.2 * Update deps: jsonpath tag v2.0.2 * Multipath: handle default arguments (#503) * Multipath: arrlen - handle missing path * Multipath: handle/add tests for default args (strlen objkeys objlen resp get strappend arrpop) (cherry picked from commit 8733934) * QA automation update (#505) (cherry picked from commit 223316f) * system-setup: added binutils for macOS (objcopy) (#524) (cherry picked from commit e6da70b) * [2.0] Cherry-pick #524, updated readies * Update version to 2.0.3 * Update readies * Bump version to 2.0.4 * Update commands.json file (2) (#534) (cherry picked from commit 58ea6f1) * Updated RS_VERSIONS (#536) (cherry picked from commit d7ee051) * Doc 2.0 (#538) (#540) * Document new path, JSON.GET and JSON.MGET * Add examples and update results for all commands * Update complexity information * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Add example to NUMINCRBY. Fix example in NUMMULTBY * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Post review rephrasing * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Add indexing JSON documents * Post review * Update docs/indexing_JSON.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Add indexing json doc (#543) * Doc 2.0 (#538) * Document new path, JSON.GET and JSON.MGET * Add examples and update results for all commands * Update complexity information * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Add example to NUMINCRBY. Fix example in NUMMULTBY * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Post review rephrasing * Update docs/commands.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Add indexing JSON documents * Post review * Update docs/indexing_JSON.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Add indexing json to the doc * update to latest readies Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> Co-authored-by: oshadmi <omer.shadmi@redislabs.com> * Build system updates (esp. target into BINDIR) (#547) (cherry picked from commit 09c4618) * [2.0] Updated readies * [2.0] Dockerfile: updated REDISEARCH_VERSION * ARM support (#550) (cherry picked from commit c3ed883) * Bump version to 2.0.5 * Build ARM for bionic; test with Redis 6.0 (#556) (#557) (cherry picked from commit b1c8cb4) * initial ijson commit (#563) * 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) * path.md copyedit suggestions (cherry picked from commit 57e6a87) * indexing_JSON.md copyedit suggestions (cherry picked from commit 8cf0cd4) * Update Cargo.toml (#552) Co-authored-by: Rafi Einstein <rafi@redislabs.com> (cherry picked from commit 42e6f32) * Update freebsd.yml (#565) * Update freebsd.yml (cherry picked from commit 58d6fbf) * JSON.SET full doc benchmark extensions (#568) * 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) * Bump mkdocs from 1.1.2 to 1.2.3 in /docs (#572) 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) * Add docs fo JSON.CLEAR and JSON.TOGGLE (#474) (#607) * fix #465 Add docs for JSON.CLEAR and JSON.TOGGLE * Document adding a new child with JSON.SET (#622) (#626) * Document adding a new child with JSON.SET * Restoring previous edits * Restoring previous edits (2) (cherry picked from commit cd2458d) Co-authored-by: Rafi Einstein <rafi@redislabs.com> Co-authored-by: Emmanuel Keller <74923777+emmanuelkeller@users.noreply.github.com> Co-authored-by: Guy Korland <gkorland@gmail.com> Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com> Co-authored-by: filipe oliveira <filipecosta.90@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> * revert bad merge (#668) * revert bad merge (2) (#669) * Add test to MOD2099 + update dep to latest generic_json_path (#671) * Fox JSON.CLEAR doc * Allow QA tests on a specific RS_VERSION (#682) * Build fixes, inc. Rocky Linux 8 (#685) * Build fixes, inc. Rocky Linux 8 * Use jsonpath lib with fix for issue #667 (#691) * Upgrade to latest jsonpath lib and add test for issue 667 * Fix comment * MOD-2785: Add benchmark for issue #674 (#693) * Add benchmark for issue #674 * Fix yml * Using memtier_benchmark for more granular control on json_recursive_descent_with_filter_uid_issue674 benchmark Co-authored-by: filipecosta90 <filipecosta.90@gmail.com> * Update release.json remove xenial - not supported add rhel8 * Enabled CI profilers (#692) * Enabled CI profilers * Enabled dwarf call graph mode and fail-fast on CI perf Co-authored-by: Rafi Einstein <rafi@redislabs.com> Co-authored-by: Emmanuel Keller <74923777+emmanuelkeller@users.noreply.github.com> Co-authored-by: Guy Korland <gkorland@gmail.com> Co-authored-by: Rachel Elledge <86307637+rrelledge@users.noreply.github.com> Co-authored-by: filipe oliveira <filipecosta.90@gmail.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: tomerhekredis <72793005+tomerhekredis@users.noreply.github.com>
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.
No description provided.