Yichun Zhang
Yichun Zhang
@lilien1010 Please give more detailed explanation on what problem you are reporting and better, with steps of reproducing the problem on our side. Simply giving a code listing is not...
@lizhitao67116961 please do not post Chinese text on all my GitHub issues boards. I'd make GitHub a pure English place for my projects. You can join the openresty (Chinese) mailing...
@hcaihao You should always check the return values of the `set()` method call and handle any errors since this call can fail.
@hcaihao This is because the Lua `false` value maps to the redis NULL value on the protocol level, which is not supported by the redis set command at all and...
@doujiang24 Yeah, I'm fine with `tostring` when the input argument is not a string.
@yyqbuct Seems like we need to port that fix over too. Pull requests welcome!
@thefosk Your patch is wrong. You cannot leave Redis replies in the system read buffer because that way the current Redis connection will be blocked by accumulated unread data. The...
@thefosk It's still wrong because sock:receive(0) does not consume any data in the system socket receive buffers, which could still prevent the remote redis server from sending more replies, leading...
@ubaldus I don't quite follow your patch. It does not make sense to add the `cols` table as the metatable to `rows`. It's just logically wrong.
@ubaldus I still do not understand your intention. Maybe you should elaborate with concrete examples?