Add RedisHashSlot by fabianfett · Pull Request #91 · swift-server/RediStack
Navigation Menu
{{ message }}
swift-server / RediStack Public
- Notifications You must be signed in to change notification settings
- Fork 30
Merged
fabianfett merged 3 commits intoswift-server:mainfrom
Sep 18, 2023Merged
fabianfett merged 3 commits intoswift-server:mainfrom
fabianfett merged 3 commits intoswift-server:mainfrom
Conversation
Copy link Copy Markdown
Member
fabianfett
commented
Sep 15, 2023
fabianfett
commented
To support Redis Cluster mode (#90), we first need a type to calculate Redis Hash Slots.
fabianfett added 2 commits
September 15, 2023 11:07
fabianfett
requested a review
from Joannis
Joannis approved these changes Sep 16, 2023
Copy link Copy Markdown
Member
Joannis
left a comment
Joannis
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, just one small question
Sources/RediStack/RedisHashSlot.swift
Outdated
Show resolved
Hide resolved
Sources/RediStack/RedisHashSlot.swift Outdated Show resolved Hide resolved
fabianfett
force-pushed
the
ff-add-RedisHashSlot
branch
from
3f24be1 to
1415d22
Compare
Joannis approved these changes Sep 18, 2023
fabianfett
merged commit
41b7777
into
swift-server:main
fabianfett
deleted the
ff-add-RedisHashSlot
branch
fabianfett
added this to the
Redis Cluster support milestone
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment