-
Notifications
You must be signed in to change notification settings - Fork 62
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
Go: Implement Pfadd, Pfcount and Select commands #2822
base: main
Are you sure you want to change the base?
Go: Implement Pfadd, Pfcount and Select commands #2822
Conversation
Signed-off-by: Niharika Bhavaraju <nbhavaraju@google.com>
Signed-off-by: Niharika Bhavaraju <nbhavaraju@google.com>
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 resolve the comments. Rest LG.
Signed-off-by: Niharika Bhavaraju <nbhavaraju@google.com>
return handleLongResponse(result) | ||
} | ||
|
||
func (client *baseClient) Select(index int64) (Result[string], error) { |
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.
This must be implemented in standalone client only. Cluster has no databases to switch into, so this command is no-op there.
// HyperLogLogCommands defines an interface for the "HyperLogLog Commands" group of commands for standalone and cluster | ||
// clients. | ||
// | ||
// [valkey.io]: https://valkey.io/commands/?group=string#hyperloglog |
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 fix the link and align the doc with #2844
Implementing the Pfadd, Pfcount and Select commands in GO client.