twemproxy icon indicating copy to clipboard operation
twemproxy copied to clipboard

redis scan command support

Open lukemakeit opened this issue 2 years ago • 5 comments

Problem

redis scan command not support.

Solution

image

Note: server_idx is saved by the lowest 12 bits of cursor, so the number of servers of the twemproxy must be less than 4096.

Result

1.1.1.1:60100> scan 0
1) "1"
2) 1) "d"
   2) "b"
   3) "c"
   4) "a"
1.1.1.1:60100> scan 1
1) "0"
2) 1) "a"
   2) "b"

lukemakeit avatar Jan 04 '23 11:01 lukemakeit

Hi @wy-ei , I have re-committed according to your suggestion, thanks.

lukemakeit avatar Jan 09 '23 07:01 lukemakeit

@lukexwang

I have see the change you made. it's better. And I think the code can be optimized again. I have leave some more suggestions.

wy-ei avatar Jan 13 '23 14:01 wy-ei

Hi @wy-ei , thank you for your suggestion. I have re-committed.

lukemakeit avatar Jan 14 '23 09:01 lukemakeit

Hello @wy-ei , 空了辛苦看下哈 ^_^

lukemakeit avatar Feb 07 '23 07:02 lukemakeit

马晓栋已经收到您的来件,谢谢!

yuerugou54 avatar Feb 07 '23 07:02 yuerugou54