Skip to content

Conversation

@enjoy-binbin
Copy link
Contributor

multi-mem: added in redis/redis#8687
resp: added in redis/redis#9508

Also adjust the order of fields to better match the output

The current output format is (redis unstable branch):

id=3 addr=127.0.0.1:50188 laddr=127.0.0.1:6379 fd=8 name= age=7 idle=0
flags=N db=0 sub=0 psub=0 multi=-1 qbuf=26 qbuf-free=20448 argv-mem=10
multi-mem=0 obl=0 oll=0 omem=0 tot-mem=40986 events=r cmd=client|list
user=default redir=-1 resp=2

multi-mem: added in redis/redis#8687
resp: added in redis/redis#9508

Also adjust the order of fields to better match the output

The current output format is (redis unstable branch):
```
id=3 addr=127.0.0.1:50188 laddr=127.0.0.1:6379 fd=8 name= age=7 idle=0
flags=N db=0 sub=0 psub=0 multi=-1 qbuf=26 qbuf-free=20448 argv-mem=10
multi-mem=0 obl=0 oll=0 omem=0 tot-mem=40986 events=r cmd=client|list
user=default redir=-1 resp=2
```
@enjoy-binbin enjoy-binbin added the to-be-merged should probably be merged soon label Jan 12, 2022
@itamarhaber
Copy link
Member

Thanks @enjoy-binbin - merging.

@itamarhaber itamarhaber merged commit cc0427b into redis:master Jan 23, 2022
@enjoy-binbin enjoy-binbin deleted the add_fields_in_client_list branch January 23, 2022 15:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

to-be-merged should probably be merged soon

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants