ofekshenawa
f711eb0f62
feat(cluster): Implement Request and Response Policy Based Routing in Cluster Mode ( #3422 )
...
* Add search module builders and tests (#1 )
* Add search module builders and tests
* Add tests
* Use builders and Actions in more clean way
* Update search_builders.go
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com >
* Update search_builders.go
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com >
* Apply suggestions from code review
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com >
* feat(routing): add internal request/response policy enums
* feat: load the policy table in cluster client (#4 )
* feat: load the policy table in cluster client
* Remove comments
* modify Tips and command pplicy in commandInfo (#5 )
* centralize cluster command routing in osscluster_router.go and refactor osscluster.go (#6 )
* centralize cluster command routing in osscluster_router.go and refactor osscluster.go
* enalbe ci on all branches
* Add debug prints
* Add debug prints
* FIX: deal with nil policy
* FIX: fixing clusterClient process
* chore(osscluster): simplify switch case
* wip(command): ai generated clone method for commands
* feat: implement response aggregator for Redis cluster commands
* feat: implement response aggregator for Redis cluster commands
* fix: solve concurrency errors
* fix: solve concurrency errors
* return MaxRedirects settings
* remove locks from getCommandPolicy
* Handle MOVED errors more robustly, remove cluster reloading at exectutions, ennsure better routing
* Fix: supports Process hook test
* Fix: remove response aggregation for single shard commands
* Add more preformant type conversion for Cmd type
* Add router logic into processPipeline
---------
Co-authored-by: Nedyalko Dyakov <nedyalko.dyakov@gmail.com >
* remove thread debugging code
* remove thread debugging code && reject commands with policy that cannot be used in pipeline
* refactor processPipline and cmdType enum
* remove FDescribe from cluster tests
* Add tests
* fix aggregation test
* fix mget test
* fix mget test
* remove aggregateKeyedResponses
* added scaffolding for the req-resp manager
* added default policies for the search commands
* split command map into module->command
* cleanup, added logic to refresh the cache
* added reactive cache refresh
* revert cluster refresh
* fixed lint
* addresed first batch of comments
* rewrote aggregator implementations with atomic for native or nearnative primitives
* addressed more comments, fixed lint
* added batch aggregator operations
* fixed lint
* updated batch aggregator, fixed extractcommandvalue
* fixed lint
* added batching to aggregateResponses
* fixed deadlocks
* changed aggregator logic, added error params
* added preemptive return to the aggregators
* more work on the aggregators
* updated and and or aggregators
* fixed lint
* added configurable policy resolvers
* slight refactor
* removed the interface, slight refactor
* change func signature from cmdName to cmder
* added nil safety assertions
* few small refactors
* added read only policies
* removed leftover prints
* Rebased to master, resolved comnflicts
* fixed lint
* updated gha
* fixed tests, minor consistency refactor
* preallocated simple errors
* changed numeric aggregators to use float64
* speculative test fix
* Update command.go
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com >
* Update main_test.go
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com >
* Add static shard picker
* Fix nil value handling in command aggregation
* Modify the Clone method to return a shallow copy
* Add clone method to digest command
* Optimize keyless command routing to respect ShardPicker policy
* Remove MGET references
* Fix MGET aggregation to map individual values to keys across shards
* Add clone method to hybrid search commands
* Undo changes in route keyless test
* remove comments
* Add test for DisableRoutingPolicies option
* Add Routing Policies Comprehensive Test Suite and Fix multi keyed aggregation for different step
---------
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com >
Co-authored-by: Nedyalko Dyakov <nedyalko.dyakov@gmail.com >
Co-authored-by: Hristo Temelski <hristo.temelski@redis.com >
2025-11-28 11:46:23 +02:00
fukua95
eb40ac8328
perf: reduce unnecessary memory allocation ( #3399 )
...
Signed-off-by: fukua95 <fukua95@gmail.com >
Co-authored-by: Nedyalko Dyakov <1547186+ndyakov@users.noreply.github.com >
2025-06-09 11:59:58 +03:00
ofekshenawa
8b1073d2d6
Support Probabilistic commands with RESP 2 protocol ( #3176 )
...
* Support bloom resp 2
* Support Resp2 for BF.Info
* simplify BFInfoCmd field assignment using map-based key-to-field references
2024-11-13 11:15:19 +02:00
Nic Gibson
84f46c3301
BUG: BFReserveArgs - error_rate & capacity ( #2763 )
...
the error_rate and capacity parameters should not be
optional - corrected
2023-10-30 18:08:06 +02:00
Tiago Peczenyj
934c6a3fe0
make public probabilistic and redis gears interfaces ( #2695 )
...
Co-authored-by: ofekshenawa <104765379+ofekshenawa@users.noreply.github.com >
2023-09-20 14:54:50 +03:00
Tiago Peczenyj
e8ad794e96
Format code and fix go vet ( #2696 )
...
* run go fix ./...
Signed-off-by: Tiago Peczenyj <tpeczenyj@weborama.com >
* run make fmt
Signed-off-by: Tiago Peczenyj <tpeczenyj@weborama.com >
* fix go vet ./... issues
* Update README.md
Reorder imports with the rules defined in the Makefile
as if we run `make fmt`
* run gofumpt -w .
* update Makefile to use gofumpt instead gofmt
* increment makefile
* format test
* format tests
Signed-off-by: Tiago Peczenyj <tpeczenyj@weborama.com >
---------
Signed-off-by: Tiago Peczenyj <tpeczenyj@weborama.com >
Co-authored-by: ofekshenawa <104765379+ofekshenawa@users.noreply.github.com >
2023-09-20 14:03:44 +03:00
ofekshenawa
33edd3de68
Rename probablistic commands with args ( #2701 )
...
Co-authored-by: Chayim <chayim@users.noreply.github.com >
2023-09-20 10:42:09 +03:00
ofekshenawa
558581eac6
bloom support ( #2673 )
2023-08-15 10:19:19 +03:00
