Added support for Redis JSON module commands, added JSON.SET and JSON.GET commands #868

vladvildanov · 2022-12-20T13:11:26Z

New functionality implemented:

Changed CI. Added one more workflow to run tests over redis-stack instances. Redis JSON as many other modules available in redis-stack which is based on redis-server >= 6.2.0. So now all redis-server instances <= 6.2.0 runs within "Redis server tests" workflow and >= 6.2.0 runs over "Redis stack tests" workflow.
Added new configuration object (for now can be used as configuration layer), that keeps information about modules.
Added new layer for command resolving, which is responsible for resolving Redis command objects from virtual methods of client. More logic to command resolving required as new module support was introduced, so it's make sense to keep it at separate layer.
Added new directory structure for module commands. Since, new module support was introduced it's make sense to keep module commands in separate directories (f.e .../Command/Redis/Json/...), so it have more structured format. New module command directories should have the same name as module (keeps in configuration) and command ID should starts with corresponding module prefix, same as in Redis (f.e Json - JSON, Search - FT etc.)
New decorator. Added new requiresRedisJsonVersion annotation, so we can mark tests to run only on instances with corresponding version of Redis JSON.
Added support for base commands JSON.GET and JSON.SET

@dvora-h @sazzad16

…tConfiguration object

…t-set-commands

…edis into vv-json-get-set-commands

.github/workflows/redis-stack-tests.yml

src/ClientConfiguration.php

tests/Predis/Command/Redis/Json/JSONSET_Test.php

tests/Predis/Command/Resolver/CommandResolverTest.php

src/Command/Traits/Json/NxXxSubcommand.php

chayim

Functionality met :D

src/Command/Traits/Json/NxXxSubcommand.php

chayim · 2023-01-09T08:21:33Z

@tillkruss what do you think of this direction? I'm happy to review the other PRs and push through - but wanted your confirmation on this.

I'm thinking post 2.1 (2.2?) we can start to roll all of these in. WDYT?

tillkruss · 2023-01-09T16:52:17Z

@tillkruss what do you think of this direction?

What do you mean by direction?

I'm thinking post 2.1 (2.2?) we can start to roll all of these in. WDYT?

I rather keep 2.1 smaller, without these larger changes.

vladvildanov · 2023-01-17T07:50:53Z

.github/workflows/redis-server-tests.yml

@@ -24,8 +24,6 @@ jobs:
          - 3
          - 4
          - 5
-          - 6


@tillkruss We don't need this versions in this workflow, we're running tests through latest version of server within our redis-stack workflow, so we make sure that we have all modules available and we can run tests for module commands

I rather keep both.

tillkruss · 2023-01-17T16:23:28Z

.github/workflows/redis-server-tests.yml

@@ -24,8 +24,6 @@ jobs:
          - 3
          - 4
          - 5
-          - 6


I rather keep both.

tillkruss · 2023-01-17T20:44:07Z

.github/workflows/redis-server-tests.yml

@@ -1,4 +1,4 @@
-name: Tests
+name: Redis_server_tests


No need to change this.

tillkruss · 2023-01-17T20:44:26Z

.github/workflows/redis-stack-tests.yml

@@ -0,0 +1,52 @@
+name: Redis_stack_tests


This file isn't needed anymore.

So we're changing original workflow to run tests over redis-stack instead of redis-server?

Keep tests.yml and call this stack.yml.

tillkruss · 2023-01-17T20:44:56Z

README.md

@@ -491,7 +491,7 @@ The code for Predis is distributed under the terms of the MIT license (see [LICE
 [ico-version-stable]: https://img.shields.io/packagist/v/predis/predis.svg?style=flat-square
 [ico-version-dev]: https://img.shields.io/packagist/vpre/predis/predis.svg?style=flat-square
 [ico-downloads-monthly]: https://img.shields.io/packagist/dm/predis/predis.svg?style=flat-square
-[ico-build]: https://img.shields.io/github/actions/workflow/status/predis/predis/tests.yml?branch=main&style=flat-square
+[ico-build]: https://img.shields.io/github/actions/workflow/status/predis/predis/redis-server-tests.yml?branch=main&style=flat-square


Let's not rename the file, it messes with GHA.

tillkruss · 2023-01-17T20:46:59Z

src/Command/RedisFactory.php

-    public function __construct()
+    private $commandResolver;
+
+    public function __construct(CommandResolverInterface $commandResolver)


This is a breaking change, can we avoid that?

Logic of command resolving becomes more complex since we adding support for modules, so In my opinion to avoid violation of SRP it makes sense to separate logic of object creation (factory) and resolving object (resolver)

We can't break the interface in a point release, we can make this a 3.0 and release when it's feature complete with Redis Stack support?

Makes sense to me. @chayim What do you think?

…get-set-commands

* Added support for Redis JSON module commands, added JSON.SET and JSON.GET commands (#868) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * add Redis stack tests * don't run tests twice * use * Update stack.yml * Rename workflows * Added version 6.x to workflow * Removed exception thrown to avoid version bug * Resolve conflicts within tests.yml * Codestyle fixes * Removew trailing whitespaces Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Added support for Bloom Filters module, added support for BF.ADD, BF.EXISTS commands (#874) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Fixed bug with incorrect tests skip * Changed module name according to original naming * Revert some old changes * Fixed misspelling Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Added support for Cuckoo Filter, added CF.ADD and CF.EXISTS commands (#907) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Fixed bug with incorrect tests skip * Changed module name according to original naming * Added support for CF.ADD and CF.EXISTS commands * Removed old files * Revert old changes Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended Bloom filters by implementing BF.INFO command (#886) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Added support for BF.INFO command * Fixed arguments data provider * Fixed bug with incorrect tests skip * Added command description * Removed unused import * Removed old directory Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extend Bloom filters by imlplementing BFMADD and BFMEXISTS commands (#887) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Fixed bug with incorrect tests skip * Added support for BF.MADD and BF.MEXISTS commands * Removed old file * Revert changes Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended Bloom Filters by implementing BF.RESERVE command (#888) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Added support for BF.INFO command * Fixed arguments data provider * Fixed bug with incorrect tests skip * Added support for BF.RESERVE command * Added command description * Codestyle fixes * Added missing test Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended Bloom Filter by implementing BF.SCANDUMP command (#889) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Fixed bug with incorrect tests skip * Added support for BF.SCANDUMP command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended Bloom Filters by implementing BF.LOADCHUNK command (#890) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Fixed bug with incorrect tests skip * Added support for BF.SCANDUMP command * Added support for BF.LOADCHUNK command * Codestyle fixes * Sort alphabetical Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended Bloom Filter support by implementing BF.INSERT command (#891) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Added support for BF.ADD and BF.EXISTS commands * Added support for BF.INFO command * Fixed arguments data provider * Fixed bug with incorrect tests skip * Added support for BF.RESERVE command * Added command description * Added support for BF.INSERT command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended Redis JSON support by implementing JSON.MGET (#892) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.MGET command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.DEL command (#909) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.DEL command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.OBJKEYS command (#911) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Add support for JSON.OBJKEYS command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.OBJLEN command (#913) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.OBJLEN command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.STRAPPEND command (#915) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.STRAPPEND command * Codestyle fix Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.STRLEN command (#917) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.STRLEN command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.NUMINCRBY command (#919) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.NUMINCRBY command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.ARRAPPEND command (#924) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.ARRAPPEND command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.ARRINDEX command (#927) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.ARRINDEX command * Codestyle fies Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.ARRINSERT command (#929) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.ARRINSERT command * Revert changes Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.ARRLEN command (#931) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Add support for JSON.ARRLEN command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.ARRPOP command (#933) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.ARRPOP command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.ARRTRIM command (#936) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.ARRTRIM command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.CLEAR command (#1004) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.CLEAR command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.RESP command (#1006) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.RESP command * Removed debugger Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.TOGGLE command (#1008) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.TOGGLE command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extended RedisJson support by implementing JSON.TYPE command (#1010) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.TYPE command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Extending RedisJson support by implementing JSON.FORGET command (#1012) * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Added CommandResolver, moved resolve command logic there, added ClientConfiguration object * Fixed test, added dependecies * Added resolved command to commands array * Used aggregation approach for modules * Added decorator to check Redis JSON module version * Added support for JSON.SET and JSON.GET commands * Added separate workflow for redis-stack tests * Changed docker imange name to correct one * Fixed indentation * Added test coverage for JSON.GET command * Changed module version resolving using annotations mapping * Re-written CommandResolver test * Update ClientInterface.php * Changes to CI, readme, removed unused modules from configuration * Fixed build badge URL * Refactored annotation check to be generic for each module * Fixed bug with incorrect tests skip * Fixed naming issue with nxXx argument * Removed redundant trait * Fixed NxXxArgument test * Added support for JSON.FORGET command Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> * Add Support for CF.ADDNX (#1048) * add support for CF.ADDNX * fix key name * fix wrong command * Added support for CF.COUNT command (#1052) Co-authored-by: Vladyslav Vildanov <[email protected]> * Support CF.MEXISTS Command (#1081) * add support for CF.ADDNX * fix key name * fix wrong command * Support CF.MEXISTS Command * fix test * Add more assertions * Added support for CF.INFO command (#1082) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for CF.DEL command (#1080) Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended BloomFilters support by implementing CF.INSERT command (#1083) * Added support for CF.INSERT command * Override argument position offset value to default one * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended CuckooFilters support by implementing CF.RESERVE command (#1086) * Added support for CF.RESERVE command * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for CF.LOADCHUNK, CF.SCANDUMP commands (#1085) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for CF.INSERTNX command (#1087) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for Count-Min Sketch with base commands CMS.INITBYDIM, CMS.INFO (#1088) * Added support for Count-Min Sketches submodule, added support for CMS.INITBYDIM adn CMS.INFO commands * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended Count-Min Sketch support by implementing CMS.INITBYPROB command (#1090) * Added support for CMS.INITBYPROB command * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended Count-Min Sketch support by implementing CMS.INCRBY and CMS.QUERY commands (#1091) * Added support for CMS.INCRBY and CMS.QUERY commands * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended Count-Min Sketch support by implementing CMS.MERGE command (#1092) * Added support for CMS.MERGE command * Added command description --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TDigest sketch, added TDIGEST.CREATE, TDIGEST.INFO commands (#1093) Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended TDigest Sketch support by implementing TDIGEST.ADD command (#1094) * Added support for TDIGEST.ADD command * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TDIGEST.BYRANK command (#1096) Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended TDigest support by implementing TDIGEST.BYREVRANK command (#1097) * Added support for TDIGEST.BYREVRANK command * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TDIGEST.CDF command (#1098) Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended support by implementing TDIGEST.MAX command (#1099) * Added support for TDIGEST.MAX command * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended support by implementing TDIGEST.MIN command (#1100) * add support for CF.ADDNX * fix key name * fix wrong command * Pulling changes * Support TDIGEST.MIN * Add support for TDIGEST.RANK command (#1103) Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended TDigest support by implementing TDIGEST.RESET command (#1104) * Added support for TDIGEST.RESET command * Codestyle fixes --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended support by implementing TDIGEST.QUANTILE command (#1101) * add support for CF.ADDNX * fix key name * fix wrong command * Pulling changes * Support TDIGEST.QUANTILE * fix returned value * fix test * fix linter error * add not exsist key test * fixes after review * fix linter problem * Add support for TDIGEST.REVRANK command (#1105) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TDIGEST.TRIMMED_MEAN command (#1106) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TOPK.RESERVE, TOPK.INFO commands (#1107) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added commands support (#1108) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TOPK.INCRBY command (#1109) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TOPK.QUERY command (#1110) Co-authored-by: Vladyslav Vildanov <[email protected]> * Added support for TDIGEST.MERGE command (#1102) Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: shacharPash <[email protected]> * Codestyle fixes * Changed return annotation * Sync branch with 2.x (#1169) * Added support for new arguments for BITPOS, BITCOUNT commands (#1045) * Added support for new arguments for EXPIRE, EXPIREAT commands (#1046) * Extended core support by implementing SORT_RO command (#1044) * Added support for SORT_RO command * Codestyle fixes * Added command description --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * fix deprecated call * Added support for container commands (#1049) * Added support for container commands FUNCTION LOAD, FUNCTION DELETE and FCALL * Changed ContainerInterface and AbstractContainer * Re-implement logic of abstract methods --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added stream commands to KeyPrefixProcessor (#1051) Co-authored-by: Vladyslav Vildanov <[email protected]> * Fix return type of ReplicationInterface::getSlaves (#1111) * Codestyle fixes * Changed return annotation --------- Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> Co-authored-by: Stephan <[email protected]> * Revert "Sync branch with 2.x (#1169)" (#1170) This reverts commit bc5f07daf906c925aea03ed01507ebe2abc02dd2. * Sync branch with 2.x (#1171) * Added support for new arguments for BITPOS, BITCOUNT commands (#1045) * Added support for new arguments for EXPIRE, EXPIREAT commands (#1046) * Extended core support by implementing SORT_RO command (#1044) * Added support for SORT_RO command * Codestyle fixes * Added command description --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * fix deprecated call * Added support for container commands (#1049) * Added support for container commands FUNCTION LOAD, FUNCTION DELETE and FCALL * Changed ContainerInterface and AbstractContainer * Re-implement logic of abstract methods --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added stream commands to KeyPrefixProcessor (#1051) Co-authored-by: Vladyslav Vildanov <[email protected]> * Fix return type of ReplicationInterface::getSlaves (#1111) * Codestyle fixes * Changed return annotation --------- Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> Co-authored-by: Stephan <[email protected]> * Container commands naming fixes (#1172) * Added support for new arguments for BITPOS, BITCOUNT commands (#1045) * Added support for new arguments for EXPIRE, EXPIREAT commands (#1046) * Extended core support by implementing SORT_RO command (#1044) * Added support for SORT_RO command * Codestyle fixes * Added command description --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * fix deprecated call * Added support for container commands (#1049) * Added support for container commands FUNCTION LOAD, FUNCTION DELETE and FCALL * Changed ContainerInterface and AbstractContainer * Re-implement logic of abstract methods --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added stream commands to KeyPrefixProcessor (#1051) Co-authored-by: Vladyslav Vildanov <[email protected]> * Fix return type of ReplicationInterface::getSlaves (#1111) * Codestyle fixes * Changed return annotation * Changed container commands names into uppercase * Changed container commands names into uppercase --------- Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> Co-authored-by: Stephan <[email protected]> * Added support for RediSearch, implemented FT.CREATE command (#1143) * Added support for new arguments for BITPOS, BITCOUNT commands (#1045) * Added support for new arguments for EXPIRE, EXPIREAT commands (#1046) * add support for CF.ADDNX * Extended core support by implementing SORT_RO command (#1044) * Added support for SORT_RO command * Codestyle fixes * Added command description --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * fix deprecated call * fix key name * fix wrong command * Added support for container commands (#1049) * Added support for container commands FUNCTION LOAD, FUNCTION DELETE and FCALL * Changed ContainerInterface and AbstractContainer * Re-implement logic of abstract methods --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added stream commands to KeyPrefixProcessor (#1051) Co-authored-by: Vladyslav Vildanov <[email protected]> * Pulling changes * Fix return type of ReplicationInterface::getSlaves (#1111) * Added support for FT.CREATE command * Fixed tests to choose correct DB * Added test coverage * Revert changes for missing commands * Added data types enums, added methods default assignments * Fixed vector field, removed default assignments, fixed tests * Added constants enum for Sortable argument, renamed arguments object * Codestyle fixes * Rename test class * Codestyle fixes * Changed return annotation * Separate common and create interfaces * Codestyle fixes * Revert changes --------- Co-authored-by: shacharPash <[email protected]> Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> Co-authored-by: Stephan <[email protected]> * Extended RediSearch support by implementing FT.SEARCH command (#1145) * Added support for new arguments for BITPOS, BITCOUNT commands (#1045) * Added support for new arguments for EXPIRE, EXPIREAT commands (#1046) * add support for CF.ADDNX * Extended core support by implementing SORT_RO command (#1044) * Added support for SORT_RO command * Codestyle fixes * Added command description --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * fix deprecated call * fix key name * fix wrong command * Added support for container commands (#1049) * Added support for container commands FUNCTION LOAD, FUNCTION DELETE and FCALL * Changed ContainerInterface and AbstractContainer * Re-implement logic of abstract methods --------- Co-authored-by: Vladyslav Vildanov <[email protected]> * Added stream commands to KeyPrefixProcessor (#1051) Co-authored-by: Vladyslav Vildanov <[email protected]> * Pulling changes * Fix return type of ReplicationInterface::getSlaves (#1111) * Added support for FT.CREATE command * Fixed tests to choose correct DB * Added test coverage * Revert changes for missing commands * Added data types enums, added methods default assignments * Added support for search command arguments, implemented FT.SEARCH command * Fixed vector field, removed default assignments, fixed tests * Tests fixes * Added constants enum for Sortable argument, renamed arguments object * Codestyle fixes * Rename test class * Merge conflicts * Codestyle fixes * Changed return annotation * Separate common and create interfaces * Codestyle fixes * Changed search arguments to condiional * Codestyle fixes --------- Co-authored-by: shacharPash <[email protected]> Co-authored-by: Vladyslav Vildanov <[email protected]> Co-authored-by: Till Krüss <[email protected]> Co-authored-by: Stephan <[email protected]> * Extended RediSearch support by implementing FT.INFO command (#1148) * add support for CF.ADDNX * fix key name * fix wrong command * Pulling changes * Added support for FT.CREATE command * Fixed tests to choose correct DB * Added test coverage * Revert changes for missing commands * Added data types enums, added methods default assignments * Added support for FT.INFO command * Codestyle fixes * Fixed vector field, removed default assignments, fixed tests * Added constants enum for Sortable argument, renamed arguments object * Codestyle fixes * Rename test class --------- Co-authored-by: shacharPash <[email protected]> Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended RediSearch support by implementing FT.ALIASADD command (#1150) * add support for CF.ADDNX * fix key name * fix wrong command * Pulling changes * Added support for FT.CREATE command * Fixed tests to choose correct DB * Added test coverage * Revert changes for missing commands * Added data types enums, added methods default assignments * Added support for FT.ALIASADD command * Fixed expected exception message * Fixed vector field, removed default assignments, fixed tests * Added constants enum for Sortable argument, renamed arguments object * Codestyle fixes * Rename test class --------- Co-authored-by: shacharPash <[email protected]> Co-authored-by: Vladyslav Vildanov <[email protected]> * Extended RediSearch support by implementing FT.ALIASDEL command (#1151) * add support for CF.ADDNX * fix key name * fix wrong command * Pulling changes * Added support for FT.CREATE command * Fixed tests to choose correct DB * Added test coverage * Revert changes for missing commands * Added data types enums, added methods default assignments * Added support for FT.ALIASADD command * Fixed expected exception message * Fixed vector field, removed default assignments, fixed tests * Add…

Vladyslav Vildanov added 11 commits December 16, 2022 18:26

Added CommandResolver, moved resolve command logic there, added Clien…

58fd299

…tConfiguration object

Fixed test, added dependecies

796e6c9

Added resolved command to commands array

d65124a

Used aggregation approach for modules

992fe77

Added decorator to check Redis JSON module version

e45c0a4

Added support for JSON.SET and JSON.GET commands

5acb81b

Added separate workflow for redis-stack tests

a706d98

Changed docker imange name to correct one

fbb3398

Fixed indentation

d7752af

Added test coverage for JSON.GET command

c855dae

Changed module version resolving using annotations mapping

67d2ca0

vladvildanov added the feature label Dec 20, 2022

vladvildanov requested review from tillkruss and chayim December 20, 2022 13:11

Vladyslav Vildanov and others added 4 commits December 20, 2022 15:26

Re-written CommandResolver test

ff17da9

Update ClientInterface.php

63ff086

Merge branch 'main' of github.com:vladvildanov/predis into vv-json-ge…

66deb79

…t-set-commands

Merge branch 'vv-json-get-set-commands' of github.com:vladvildanov/pr…

92de663

…edis into vv-json-get-set-commands