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.

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

async cluster: Group responses by response_policy. #888

Merged
merged 2 commits into from
Aug 16, 2023

Conversation

nihohit
Copy link
Contributor

@nihohit nihohit commented Jul 14, 2023

This change implements the response_policy tip in the async cluster.
https://redis.io/docs/reference/command-tips/#response_policy

This means that the results from fan-out commands will be aggregated
differently, based on the sent command.

@nihohit nihohit force-pushed the response-policy branch 2 times, most recently from 4be21ce to c1d7211 Compare July 17, 2023 11:15
nihohit added a commit to amazon-contributing/redis-rs that referenced this pull request Jul 17, 2023
@nihohit nihohit force-pushed the response-policy branch 2 times, most recently from 14084dc to 9fead2a Compare July 23, 2023 05:14
@jaymell
Copy link
Contributor

jaymell commented Aug 16, 2023

Looks great!

This change implements the response_policy tip in the async cluster.
https://redis.io/docs/reference/command-tips/#response_policy

This means that the results from fan-out commands will be aggregated
differently, based on the sent command.
@nihohit nihohit merged commit 7643f37 into redis-rs:main Aug 16, 2023
10 checks passed
@nihohit nihohit deleted the response-policy branch August 16, 2023 19:23
altanozlu pushed a commit to altanozlu/redis-rs that referenced this pull request Aug 16, 2023
* Add NOTBUSY error code.

* async cluster: Group responses by response_policy.

This change implements the response_policy tip in the async cluster.
https://redis.io/docs/reference/command-tips/#response_policy

This means that the results from fan-out commands will be aggregated
differently, based on the sent command.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants