[2.4] MOD-4850: Merge all from master towards v2.4.5 by oshadmi · Pull Request #927 · RedisJSON/RedisJSON

oshadmi · 2023-02-12T09:28:22Z

No description provided.

* Add to JSONPath filter the regexp match operator * Improve coverage * Minor cosmetics * Allow match using regex pattern from a field and add end-to-end test * Test with numeric combined in filter * Add more tests and documentation

* Add details on jsonpath filter * fix local path and some edits * Apply suggestions from review by Nermina Co-authored-by: Nermina Miller <102551568+nermiller@users.noreply.github.com> Co-authored-by: Nermina Miller <102551568+nermiller@users.noreply.github.com>

Variable fix. <s>`$search`</s> into `$rejson` ``` rejson=$(docker run -d -it -v $PWD:/build debian:bullseye bash) docker exec -it $rejson bash ``` Co-authored-by: Guy Korland <gkorland@gmail.com>

Co-authored-by: Guy Korland <gkorland@gmail.com>

* Partial fix for REDISIO-186: JSON broken links * Applied feedback from Chayim * Updates examples and restructures sections * Update docs/commands/json.arrappend.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrappend.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrindex.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrindex.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrinsert.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrindex.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrindex.md * Update docs/commands/json.arrinsert.md * Update docs/commands/json.arrlen.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrpop.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrtrim.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.numincrby.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.resp.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.set.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.strappend.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.toggle.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrpop.md * Update docs/commands/json.arrpop.md * Update docs/commands/json.arrinsert.md * Update json.arrtrim.md * Update docs/commands/json.arrappend.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrinsert.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrtrim.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.arrpop.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com>

* License change * License hyperlinks in README.md * RAMP file fix * Cargo.toml fix * Fix spelling wordlist

* Support legacy jsonpath with dollar * Support legacy jsonpath with dollar * Fix test

* Fixes #882, explains returns of JSON array and JSON object * Update docs/commands/json.get.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> * Update docs/commands/json.get.md Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com> Co-authored-by: Omer Shadmi <76992134+oshadmi@users.noreply.github.com>

* Update README.md

* add support for arrindex for none scalars * test no scalar on arrindex * clean code rename args * fix tests * fix tests

* clean clippy warnings * revert so we can build with Rust 1.60

Co-authored-by: Guy Korland <gkorland@gmail.com>

add focal CI testing on enterprise

Update qatests

* enable tracing jsonpath tests * add flow test * Evaluate all filter operands * Add tests with precedence and nesting * update spellcheck dictionary * remove redis-module from dev-dependencies

add cluster versions for release testing

Update RS_VERSIONS

…rformance nightly only once a week (#896)

* update to the latest stable rust even if rust installed * update readies

* straight the error message * fix build * format code

* Added `rustis` client library [rustis](https://github.com/dahomey-technologies/rustis) * Update README.md Added rustis client library * Update README.md Added rustis documentation link Co-authored-by: Guy Korland <gkorland@gmail.com>

* break serde-json manager out to a file * add license header

) * Enabled Pull Request CI Performance info generation and publishing * Updated label triggering to ensure default flow is not canceled

* ignore case on spelling * plugin version bump * spelling

* disable recursion limit on RDB load

…o_2.4_20230212

codecov · 2023-02-12T09:45:46Z

Codecov Report

Base: 81.40% // Head: 81.40% // No change to project coverage 👍

Coverage data is based on head (7356886) compared to base (eb5eba8).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files

@@           Coverage Diff           @@
##              2.4     #927   +/-   ##
=======================================
  Coverage   81.40%   81.40%           
=======================================
  Files          15       15           
  Lines        3905     3905           
=======================================
  Hits         3179     3179           
  Misses        726      726

Impacted Files	Coverage Δ
src/jsonpath/mod.rs	`85.71% <ø> (ø)`
src/lib.rs	`86.53% <100.00%> (ø)`

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

MeirShpilraien

👍
Reviewed the code changes, @rafie please review the build changes.

oshadmi and others added 30 commits November 6, 2022 10:17

Allow repetition of filter relation instead of optioanl (#850)

ce225df

Build and automation updates (#853)

ece58d3

Update developer.md (#857)

c0bf0ab

Variable fix. <s>`$search`</s> into `$rejson` ``` rejson=$(docker run -d -it -v $PWD:/build debian:bullseye bash) docker exec -it $rejson bash ``` Co-authored-by: Guy Korland <gkorland@gmail.com>

Fixes link (#855)

874d4c0

Co-authored-by: Guy Korland <gkorland@gmail.com>

Updated readies & more nightly builds (#860)

33de90f

License change (#864)

099f9ce

* License change * License hyperlinks in README.md * RAMP file fix * Cargo.toml fix * Fix spelling wordlist

Bump latest JSON API version to RedisJSON_V3 (#867)

d4e4e3a

MOD-4436: Support legacy jsonpath with dollar (#861)

8583f7f

* Support legacy jsonpath with dollar * Support legacy jsonpath with dollar * Fix test

Update dependencies (cargo; readies) (#871)

2683396

Fix json.debug malformed link (#870)

1153675

Clarify when filter regex match was introduced (#878)

f43dfb8

Add run with docker (#888)

7d0ceb3

Redisson client added to the list (#891)

4412739

* Update README.md

Allow JSON.ARRINDEX with none scalar values (#892)

6ff8be9

* add support for arrindex for none scalars * test no scalar on arrindex * clean code rename args * fix tests * fix tests

clean clippy warnings (#847)

9b39a0c

* clean clippy warnings * revert so we can build with Rust 1.60

Removing LGTM as the service is going offline shortly (#874)

38bf9fc

Co-authored-by: Guy Korland <gkorland@gmail.com>

Update qatests

6b979ef

add focal CI testing on enterprise

Merge pull request #893 from RedisJSON/TomerHekmati-patch-1

1a978ec

Update qatests

MOD-4602: handle jsonpath filter with 3 or more operands (#890)

6b30093

* enable tracing jsonpath tests * add flow test * Evaluate all filter operands * Add tests with precedence and nesting * update spellcheck dictionary * remove redis-module from dev-dependencies

Update RS_VERSIONS

32e34f5

add cluster versions for release testing

Merge pull request #895 from RedisJSON/TomerHekmati-patch-1

6bddd09

Update RS_VERSIONS

Updated benchmark trigger by label action:run-benchmark and to run pe…

70c0bf3

…rformance nightly only once a week (#896)

avoid coyback to host (#899)

57cf7ba

Replace wrong word 'subcommands' with 'arguments' (#900)

e3a9e37

Updating the dockerhub badge to redis-stack-server (#901)

be2fd89

Updated dependencies (#903)

7fcba35

gkorland and others added 11 commits December 29, 2022 16:17

update to the latest stable rust even if rust installed (#898)

1b381b7

* update to the latest stable rust even if rust installed * update readies

Fix #725: straight the error messages (#813)

b3ea215

* straight the error message * fix build * format code

Added rustis client library (#904)

18ad25a

* Added `rustis` client library [rustis](https://github.com/dahomey-technologies/rustis) * Update README.md Added rustis client library * Update README.md Added rustis documentation link Co-authored-by: Guy Korland <gkorland@gmail.com>

break serde-json manager out to a file (#910)

5b6a854

* break serde-json manager out to a file * add license header

Enabled Pull Request CI Performance info generation and publishing (#911

078ac5d

) * Enabled Pull Request CI Performance info generation and publishing * Updated label triggering to ensure default flow is not canceled

Ignore case on spell check (#914)

b458d28

* ignore case on spelling * plugin version bump * spelling

Add ipv6 capability (#918)

54125ef

MOD-4822: Disable recursion limit on RDB load (#919)

5f701d0

* disable recursion limit on RDB load

Build updates (#922)

ed7b5da

Use Rust toolchain v1.66.1 for macOS/x64 due to compiler issue (#923)

9ec5c41

Merge remote-tracking branch 'origin/master' into oshadmi_cp_master_t…

f4b60b7

…o_2.4_20230212

update version to 2.4.5

7356886

oshadmi requested a review from MeirShpilraien February 12, 2023 09:55

oshadmi merged commit 4e80773 into 2.4 Feb 12, 2023

oshadmi deleted the oshadmi_cp_master_to_2.4_20230212 branch February 12, 2023 16:59

oshadmi changed the title ~~[2.4] Merge all from master towards v2.4.5~~ [2.4] MOD-4850: Merge all from master towards v2.4.5 Feb 12, 2023

MeirShpilraien reviewed Feb 13, 2023

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Comments

[2.4] MOD-4850: Merge all from master towards v2.4.5#927

[2.4] MOD-4850: Merge all from master towards v2.4.5#927
oshadmi merged 42 commits into2.4from
oshadmi_cp_master_to_2.4_20230212

oshadmi commented Feb 12, 2023

Uh oh!

codecov bot commented Feb 12, 2023 •

edited

Loading

Uh oh!

MeirShpilraien left a comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

Comments

Conversation

oshadmi commented Feb 12, 2023

Uh oh!

codecov bot commented Feb 12, 2023 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

MeirShpilraien left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

13 participants

codecov bot commented Feb 12, 2023 •

edited

Loading