You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kvrocks.apache.org by GitBox <gi...@apache.org> on 2022/09/15 22:20:07 UTC

[GitHub] [incubator-kvrocks] usrtax opened a new issue, #880: add support for resp3

usrtax opened a new issue, #880:
URL: https://github.com/apache/incubator-kvrocks/issues/880

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/incubator-kvrocks/issues) and found no similar issues.
   
   
   ### Motivation
   
   https://github.com/redis/redis-specifications/blob/master/protocol/RESP3.md
   RESP3 is an updated version of RESP v2, which is the protocol used in Redis starting with roughly version 2.0 (1.2 already supported it, but Redis 2.0 was the first version to talk only this protocol). The name of this protocol is just RESP3 and not RESP v3 or RESP 3.0.
   
   
   
   
   ### Solution
   
   _No response_
   
   ### Are you willing to submit a PR?
   
   - [ ] I'm willing to submit a PR!


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

Re: [I] Add support for RESP3 [kvrocks]

Posted by "git-hulk (via GitHub)" <gi...@apache.org>.
git-hulk closed issue #880: Add support for RESP3
URL: https://github.com/apache/kvrocks/issues/880


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] git-hulk commented on issue #880: Add support for RESP3

Posted by "git-hulk (via GitHub)" <gi...@apache.org>.
git-hulk commented on issue #880:
URL: https://github.com/apache/incubator-kvrocks/issues/880#issuecomment-1584234135

   > We can give a tracking issue based on different types from [resp3.md](https://github.com/redis/redis-specifications/blob/master/protocol/RESP3.md) currently.
   
   Sure, it's good to separate sub-tasks by the data type, but we need to offer a unified way to determine whether to use RESP3 or not like the encoding: https://github.com/apache/incubator-kvrocks/blob/f557255cec8d4f43cf63144b712c71026bc9320e/CMakeLists.txt#L34. And should NOT enable it before all data types are supported RESP3.
   
   
   


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-kvrocks] infdahai commented on issue #880: Add support for RESP3

Posted by "infdahai (via GitHub)" <gi...@apache.org>.
infdahai commented on issue #880:
URL: https://github.com/apache/incubator-kvrocks/issues/880#issuecomment-1584061847

   BTW, we can split the task to many functions and support the resp3 protocol gradually.
   
   We can give a tracking issue based on anchors from [resp3.md](https://github.com/redis/redis-specifications/blob/master/protocol/RESP3.md) currently


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@kvrocks.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org