-
Notifications
You must be signed in to change notification settings - Fork 3.8k
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
Support REPLICAOF Command #2811
Conversation
Codecov Report
@@ Coverage Diff @@
## master #2811 +/- ##
============================================
- Coverage 58.40% 58.26% -0.15%
- Complexity 3009 3011 +2
============================================
Files 174 175 +1
Lines 10821 10849 +28
Branches 621 623 +2
============================================
+ Hits 6320 6321 +1
- Misses 4286 4314 +28
+ Partials 215 214 -1
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please deprecate SLAVEOF commands.
You can check #2673 for idea.
Also, put the new methods just after those.
REPLICAOF