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.

Sign up for GitHub

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

Jump to bottom

Ort fixup #2579

Draft
wants to merge 79 commits into
base: main
Choose a base branch
from
Draft

Ort fixup #2579

wants to merge 79 commits into from

Conversation

ikolomi
Copy link
Collaborator

@ikolomi ikolomi commented Nov 3, 2024

Issue link

This Pull Request is linked to issue (URL): [REPLACE ME]

Checklist

Before submitting the PR make sure the following are checked:

  • This Pull Request is related to one issue.
  • Commit message has a detailed description of what changed and why.
  • Tests are added or updated.
  • CHANGELOG.md and documentation files are updated.
  • Destination branch is correct - main or release
  • Commits will be squashed upon merging.

shohamazon and others added 30 commits October 10, 2024 16:09
@shohamazon
Python: adds JSON.ARRLEN command (#2403)
fcc6001 
---------

Signed-off-by: Shoham Elias <[email protected]>
Signed-off-by: Shoham Elias <[email protected]>
@shohamazon
Python: adds JSON.CLEAR command (#2418)
a2ea32e 
Signed-off-by: Shoham Elias <[email protected]>
@prateek-kumar-improving
Python [FT.CREATE] command added(Created for release-1.2 branch) (#2426)
8fea303 
* Python [FT.CREATE] command added

Signed-off-by: Prateek Kumar <[email protected]>

---------

Signed-off-by: Prateek Kumar <[email protected]>
@Yury-Fridlyand
Java: FT.CREATE (#2414)
bbfce44 
Signed-off-by: Yury-Fridlyand <[email protected]>
@GilboaAWS
Added inflightRequestsLimit client config to java (#2443)
855bb33 
Added inflightRequestsLimit client config to java (#2408)

* Add inflight request limit config to java

Signed-off-by: GilboaAWS <[email protected]>
@shohamazon
Fix CI failing (#2444)
705b404 
Signed-off-by: Shoham Elias <[email protected]>
@shohamazon
Fix mypy failing (#2453)
3661f87 
---------

Signed-off-by: Shoham Elias <[email protected]>
@eifrah-aws
Make redis-rs part of this repo (#2456)
9d953d3 
* Make redis-rs part of this repo
* Improved PYTHON DEVELOPERS.md file
* Added Makefile for unified build method for all the languages
@Muhammad-awawdi-amazon @shohamazon
Python: add JSON.NUMINCRBY command (#2448)
1b3cad3 
---------

Signed-off-by: Muhammad Awawdi <[email protected]>
Signed-off-by: Muhammad-awawdi-amazon <[email protected]>
Co-authored-by: Shoham Elias <[email protected]>
@GilboaAWS
Added inflightRequestsLimit client config to Node (#2452)
04f0910 
* Added inflightRequestsLimit client config to Node

---------

Signed-off-by: GilboaAWS <[email protected]>
@Yury-Fridlyand
Java: Allow to run modules CI on demand. (#2416)
b255189 
* Run modules CI on demand.

Signed-off-by: Yury-Fridlyand <[email protected]>
@prateek-kumar-improving
Python FT.DROPINDEX command (#2437)
23a180b 
* Python [FT.DROPINDEX] Added command

---------

Signed-off-by: Prateek Kumar <[email protected]>
@Yury-Fridlyand
Update routing for commands from server modules. (#2461)
e51aa6d 
* Update routing for commands from server modules.

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand @acarbonetto
Java: FT.DROPINDEX. (#2440)
9b9ccdb 
* `FT.DROPINDEX`.

Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
@Muhammad-awawdi-amazon @shohamazon
Python: Adds JSON.NUMMULTBY Command (#2458)
db785c9 
---------

Signed-off-by: Muhammad Awawdi <[email protected]>
Co-authored-by: Shoham Elias <[email protected]>
@eifrah-aws
redis-rs code cleanup: keep a single async runtime: tokio (#2459)
15d57b2 
Keep a single Runtime: `tokio`

- Removed dead code for `async-std`
- Fixed `redis-rs` tests so they could be run with a simple `cargo test` command
- Changed the default features to include: "tokio-comp", "tokio-rustls-comp", "connection-manager", "cluster", "cluster-async"
- Fixed a flaky `glide-core` test

Signed-off-by: Eran Ifrah <[email protected]>
@Yury-Fridlyand
Fix python CI: fix linter installation (#2465)
1b6db03 
fix ci

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand
Java: FT.SEARCH (#2439)
c252cbb 
* `FT.CREATE`

Signed-off-by: Yury-Fridlyand <[email protected]>
@prateek-kumar-improving
Python: Add commands FT.ALIASADD, FT.ALIASDEL, FT.ALIASUPDATE (#…
1125eba 
…2471)

* Python: Add commands FT.ALIASADD, FT.ALIASDEL, FT.ALIASUPDATE

---------

Signed-off-by: Prateek Kumar <[email protected]>
@jamesx-improving
Java: add JSON.SET and JSON.GET (#2462)
adcc76f 
* Java: add JSON.SET and JSON.GET

---------

Signed-off-by: James Xin <[email protected]>
@tjzhang-BQ @Yury-Fridlyand @cyip10
Node: Add CI support for server modules (#2472)
fa12563 
* Node: add server modules CI support

---------

Signed-off-by: TJ Zhang <[email protected]>
Signed-off-by: Yury-Fridlyand <[email protected]>
Signed-off-by: Chloe <[email protected]>
Co-authored-by: TJ Zhang <[email protected]>
Co-authored-by: Yury-Fridlyand <[email protected]>
Co-authored-by: Chloe <[email protected]>
@Yury-Fridlyand
Java: FT.AGGREGATE. (#2468)
87412e8 
* `FT.AGGREGATE`.

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand
Java: add FT.INFO (#2405)
d291d87 
* `FT.INFO`

Signed-off-by: Yury-Fridlyand <[email protected]>
@cyip10 @acarbonetto
Java: FT.ALIASADD, FT.ALIASDEL, FT.ALIASUPDATE (#2442)
b168972 
Signed-off-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
@tjzhang-BQ
Node: add commands JSON.GET and JSON.SET (#2427)
91cafe8 
* Node: add commands JSON.GET and JSON.SET

Signed-off-by: TJ Zhang <[email protected]>
@Yury-Fridlyand
Java: JSON.ARRINSERT and JSON.ARRLEN (#2476)
56b4401 
* `JSON.ARRINSERT` and `JSON.ARRLEN`

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand
Run CI on PR merge on release branches (#2475)
6a3a33f 
* fix CI

Signed-off-by: Yury-Fridlyand <[email protected]>

* fix CI

Signed-off-by: Yury-Fridlyand <[email protected]>

---------

Signed-off-by: Yury-Fridlyand <[email protected]>
@barshaul
Avoid retrying on IO errors when it’s unclear if the server received …
914fd60 
…the request (#2479)

* Avoid retrying on IO errors when it’s unclear if the server received the request

Signed-off-by: barshaul <[email protected]>
@avifenesh
Rust - code cleanup, part 1 (#2478)
695697a 
Some cleaning for unnecessary code to set the floor for next PR

Signed-off-by: avifenesh <[email protected]>
@yipin-chen
Java: add JSON.ARRAPPEND command (#2489)
76f68a6 
* Java: add JSON.ARRAPPEND command
---------
Signed-off-by: Yi-Pin Chen <[email protected]>
shohamazon and others added 27 commits October 27, 2024 10:33
@shohamazon @acarbonetto @jonathanl-bq
Python: adds JSON.STRLEN, JSON.STRAPPEND commands (#2372)
8d09a1c 
---------

Signed-off-by: Shoham Elias <[email protected]>
Signed-off-by: Shoham Elias <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
Co-authored-by: jonathanl-bq <[email protected]>
@shohamazon
Python: adds JSON.OBJKEYS command (#2395)
d712983 
---------

Signed-off-by: Shoham Elias <[email protected]>
@shohamazon
Python: adds JSON.ARRINSERT command
08cce19 
Signed-off-by: Shoham Elias <[email protected]>
@shohamazon
Python: reorder json.py commands
a99d89a 
Signed-off-by: Shoham Elias <[email protected]>
@Muhammad-awawdi-amazon @shohamazon
Python: add JSON.DEBUG.FIELDS and JSON.DEBUG.MEMORY commands (#2481)
d95040c 
---------

Signed-off-by: Muhammad Awawdi <[email protected]>
Signed-off-by: Shoham Elias <[email protected]>
Co-authored-by: Shoham Elias <[email protected]>
@shohamazon
Python: adds JSON.ARRTRIM command (#2457)
c197f1d 
---------

Signed-off-by: Shoham Elias <[email protected]>
@BoazBD @shohamazon
Python - Implement JSON.OBJLEN command functionality (#2495)
52dd474 
---------

Signed-off-by: BoazBD <[email protected]>
Signed-off-by: Shoham Elias <[email protected]>
Co-authored-by: Shoham Elias <[email protected]>
@Yury-Fridlyand
Java: JSON.ARRPOP. (#2486)
98df691 
* `JSON.ARRPOP`.

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand
Java: FT.PROFILE (#2473)
cb21081 
* `FT.PROFILE`.

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand
Java: JSON.DEBUG. (#2520)
a882b1f 
* `JSON.DEBUG`.

Signed-off-by: Yury-Fridlyand <[email protected]>
@tjzhang-BQ
Node: Add command JSON.RESP (#2517)
01308db 
Node: Add command JSON.RESP

Signed-off-by: TJ Zhang <[email protected]>
@Yury-Fridlyand
CI: Minor fixes (#2333)
5cf72b1 
Update CI

Signed-off-by: Yury-Fridlyand <[email protected]>
@jonathanl-bq
Java: add JSON.STRAPPEND and JSON.STRLEN (#2522)
7db739e 
* Implement JSON STRLEN and STRAPPEND commands

---------

Signed-off-by: Jonathan Louie <[email protected]>
Signed-off-by: jonathanl-bq <[email protected]>
@jonathanl-bq
Java: add JSON.NUMINCRBY and JSON.NUMMULTBY (#2511)
5e86b7a 
* Implement NUMINCRBY and NUMMULTBY JSON commands

---------

Signed-off-by: Jonathan Louie <[email protected]>
Signed-off-by: jonathanl-bq <[email protected]>
@cyip10 @acarbonetto
Java: FT.EXPLAIN and FT.EXPLAINCLI (#2515)
ddca067 
Signed-off-by: Chloe <[email protected]>
Signed-off-by: Chloe Yip <[email protected]>
Signed-off-by: Andrew Carbonetto <[email protected]>
Co-authored-by: Andrew Carbonetto <[email protected]>
@prateek-kumar-improving
Python: FT.AGGREGATE command added (#2530)
7b5b72d 
* Python: FT.AGGREGATE command added

---------

Signed-off-by: Prateek Kumar <[email protected]>
@jamesx-improving
Java: add JSON.TYPE (#2525)
53743a9 
* Java: add JSON.TYPE

---------

Signed-off-by: James Xin <[email protected]>
@shohamazon
Python: adds JSON.ARRAPPEND command (#2382)
1e0476c 
---------

Signed-off-by: Shoham Elias <[email protected]>
Signed-off-by: Shoham Elias <[email protected]>
@shohamazon
Python: adds JSON.RESP command (#2451)
2284c75 
---------

Signed-off-by: Shoham Elias <[email protected]>
@Yury-Fridlyand
Java: Fix script kill IT (#2523)
ff39238 
* Fix script kill IT

Signed-off-by: Yury-Fridlyand <[email protected]>
@Yury-Fridlyand
Add node REPL (#2355)
4af45f8 
* Add node REPL

Signed-off-by: Yury-Fridlyand <[email protected]>
@Muhammad-awawdi-amazon @shohamazon
Python: Add JSON.ARRINDEX Command (#2528)
5b90096 
---------

Signed-off-by: Muhammad Awawdi <[email protected]>
Signed-off-by: Shoham Elias <[email protected]>
Co-authored-by: Shoham Elias <[email protected]>
@Yury-Fridlyand
Node: FT.INFO (#2540)
d4ef3c3 
* `FT.INFO`.

Signed-off-by: Yury-Fridlyand <[email protected]>
@tjzhang-BQ
Node: Add command JSON.STRLEN and JSON.STRAPPEND (#2537)
8ef77b6 
* Node: Add command JSON.STRLEN

Signed-off-by: TJ Zhang <[email protected]>
@Yury-Fridlyand
Node: JSON.ARRINSERT, JSON.ARRPOP and JSON.ARRLEN. (#2542)
6b06138 
* `JSON.ARRINSERT`, `JSON.ARRPOP` and `JSON.ARRLEN`.

Signed-off-by: Yury-Fridlyand <[email protected]>
@ikolomi
Fixes #2556 : Mutable default parameter in python example.
d598e24 
Signed-off-by: ikolomi <[email protected]>
@ikolomi
WIP
135026b
@ikolomi ikolomi requested a review from a team as a code owner November 3, 2024 09:47
@Yury-Fridlyand Yury-Fridlyand marked this pull request as draft November 4, 2024 15:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.