-
Couldn't load subscription status.
- Fork 2.5k
Implement Request and Response Policy Based Routing in Cluster Mode #3422
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.
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
Open
ofekshenawa
wants to merge
47
commits into
redis:load-balance-search-commands-to-shards
Choose a base branch
from
ofekshenawa:load-balance-search-commands-to-shards
base: load-balance-search-commands-to-shards
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 15 commits
Commits
Show all changes
47 commits
Select commit
Hold shift + click to select a range
82a3433
feat(routing): add internal request/response policy enums
ofekshenawa 9e4369a
Merge pull request #3 from ofekshenawa/define-policy-type
ofekshenawa 74407a0
feat: load the policy table in cluster client (#4)
ofekshenawa f99c63b
modify Tips and command pplicy in commandInfo (#5)
ofekshenawa b6633bf
centralize cluster command routing in osscluster_router.go and refact…
ofekshenawa ed528f8
Merge branch 'load-balance-search-commands-to-shards' into load-balan…
ofekshenawa 43fcc67
remove thread debugging code
ofekshenawa 7eb3818
remove thread debugging code && reject commands with policy that cann…
ofekshenawa de344fd
refactor processPipline and cmdType enum
ofekshenawa 57cdd32
remove FDescribe from cluster tests
ofekshenawa 04a110a
Add tests
ofekshenawa f1c7f62
fix aggregation test
ofekshenawa e0b122a
fix mget test
ofekshenawa a2ffd62
fix mget test
ofekshenawa c00bd81
remove aggregateKeyedResponses
ofekshenawa de1b16c
added scaffolding for the req-resp manager
htemelski-redis 1b2eaa6
added default policies for the search commands
htemelski-redis 64245f8
split command map into module->command
htemelski-redis 3397b6f
cleanup, added logic to refresh the cache
htemelski-redis 4fb4c68
added reactive cache refresh
htemelski-redis bd526a8
revert cluster refresh
htemelski-redis 5b01de5
fixed lint
htemelski-redis 4d1d775
updated build workflow
htemelski-redis 2a06726
update build action
htemelski-redis 17201a1
addresed first batch of comments
htemelski-redis d7f7ad3
rewrote aggregator implementations with atomic for native or nearnati…
htemelski-redis cfb290a
addressed more comments, fixed lint
htemelski-redis 62442b5
added batch aggregator operations
htemelski-redis 14bd6e1
fixed lint
htemelski-redis eedb3ed
updated batch aggregator, fixed extractcommandvalue
htemelski-redis f621126
fixed lint
htemelski-redis a909347
added batching to aggregateResponses
htemelski-redis 9b78583
fixed deadlocks
htemelski-redis 7d3ffbf
changed aggregator logic, added error params
htemelski-redis cfc5174
added preemptive return to the aggregators
htemelski-redis a1a4191
more work on the aggregators
htemelski-redis b2402d3
updated and and or aggregators
htemelski-redis d241b05
fixed lint
htemelski-redis bb81382
added configurable policy resolvers
htemelski-redis dc04bb2
updated build action
htemelski-redis 70bb290
slight refactor
htemelski-redis ec3fc78
removed the interface, slight refactor
htemelski-redis ec36741
change func signature from cmdName to cmder
htemelski-redis 87a12cd
added nil safety assertions
htemelski-redis 17ac170
few small refactors
htemelski-redis ea795e3
added read only policies
htemelski-redis a4ac8df
removed leftover prints
htemelski-redis File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.