-
Notifications
You must be signed in to change notification settings - Fork 504
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
Add CLIENT INFO command as Redis #1354
Add CLIENT INFO command as Redis #1354
Conversation
This PR add CLIENT INFO command as Redis, this command returns information about the current client connection, the output format is the same as CLIENT LIST command. CLIENT INFO was added in Redis 6.2.0
@@ -45,6 +45,11 @@ func TestIntrospection(t *testing.T) { | |||
require.Regexp(t, "id=.* addr=.*:.* fd=.* name=.* age=.* idle=.* flags=N namespace=.* qbuf=.* .*obuf=.* cmd=client.*", v) | |||
}) | |||
|
|||
t.Run("CLIENT INFO", func(t *testing.T) { | |||
v := rdb.Do(ctx, "CLIENT", "INFO").Val() |
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.
note go-redis does not support CLIENT INFO yet (a pending PR, should be supported soon)
so here i am using Do
Cool, Thanks for your contribution. @enjoy-binbin We can use the command |
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.
LGTM. @enjoy-binbin just run the command @git-hulk provided and commit changes.
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.
LGTM
Thanks all, merging.. |
This PR add CLIENT INFO command as Redis, this command
returns information about the current client connection,
the output format is the same as CLIENT LIST command.
CLIENT INFO was added in Redis 6.2.0