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

Add CLUSTER SHARDS command #2984

Closed
sazzad16 opened this issue Apr 24, 2022 · 3 comments · Fixed by #3598
Closed

Add CLUSTER SHARDS command #2984

sazzad16 opened this issue Apr 24, 2022 · 3 comments · Fixed by #3598
Labels
enhancement

Comments

@sazzad16
Copy link
Collaborator

sazzad16 commented Apr 24, 2022 •
edited
Loading

https://redis.io/commands/cluster-shards/
@yaopen4
Copy link

yaopen4 commented Apr 26, 2022

Hi!
I'd love to take this as my first issue

@sazzad16
Copy link
Collaborator Author

@yaopen4 You're welcome to give it a shot 😃

wormogig pushed a commit to wormogig/jedis that referenced this issue Oct 21, 2023
Add CLUSTER SHARDS command (redis#2984)
1f7d14c
wormogig pushed a commit to wormogig/jedis that referenced this issue Oct 26, 2023
Add CLUSTER SHARDS command (redis#2984)
c82a3c4
@wormogig
Copy link
Contributor

Hello.
I don't really understand the status of the issue and opened PR for it.
If the issue is not relevant - write, I will close it.

@sazzad16 sazzad16 linked a pull request Nov 14, 2023 that will close this issue
Add CLUSTER SHARDS command support (#2984) #3598
Merged
sazzad16 added a commit that referenced this issue Nov 14, 2023
@wormogig @sazzad16
Add CLUSTER SHARDS command support (#3598)
452ea52 
* Add CLUSTER SHARDS command (#2984)

* Added new classes for CLUSTER SHARDS response

* Slots type changed to List<List<Long>> + style fixed

* Apply suggestions from code review

* reset redis before ClusterCommandTest added

* isSsl() added to ClusterShardNodeInfo

* Several assertion added to clusterShards test

---------

Co-authored-by: Khokhlov Aleksey <wormogig@gmail.com>
Co-authored-by: M Sazzadul Hoque <7600764+sazzad16@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add CLUSTER SHARDS command support (#2984) wormogig/jedis
3 participants
@sazzad16 @wormogig @yaopen4