zyearn commented on a change in pull request #972: Redis server protocol
URL: https://github.com/apache/incubator-brpc/pull/972#discussion_r354745678
##########
File path: src/brpc/redis_reply.h
##########
@@ -160,6 +189,80 @@ inline int64_t RedisReply::integer() const {
return 0;
}
+inline bool RedisReply::SetNilString() {
+ if (!_arena) return false;
+ _type = REDIS_REPLY_STRING;
+ _length = npos;
Review comment:
用来区分 是nill string。这里不能单纯把type设置为redis_reply_nil，序列化的时候要区分是nil string 还是 nil
array
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
With regards,
Apache Git Services
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
