You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@kvrocks.apache.org by "PragmaTwice (via GitHub)" <gi...@apache.org> on 2024/02/10 13:25:27 UTC

[I] Support namespace in Pub/Sub [kvrocks]

PragmaTwice opened a new issue, #2097:
URL: https://github.com/apache/kvrocks/issues/2097

   ### Search before asking
   
   - [X] I had searched in the [issues](https://github.com/apache/kvrocks/issues) and found no similar issues.
   
   
   ### Motivation
   
   Currently the namespace mechanism is not supported by Pub/Sub series commands.
   
   Actually it may be not hard to support by using concurrent containers in TBB.
   
   ### 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] Support namespace in Pub/Sub commands [kvrocks]

Posted by "JxLi0921 (via GitHub)" <gi...@apache.org>.

JxLi0921 commented on issue #2097:
URL: https://github.com/apache/kvrocks/issues/2097#issuecomment-1940915724

   According to the redis docs( https://redis.io/docs/interact/pubsub/ ), the Pub/Sub command in redis is not related to the database numbers.
   > Pub/Sub has no relation to the key space.
   It was made to not interfere with it on any level, including database numbers.
   Publishing on db 10, will be heard by a subscriber on db 1.
   
   It seems that the namespace mechanism in kvrocks has similar purpose as the database numbers in redis, so if we want the commands to be consistent with redis, will it be better if we just keep using current implementation?


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

Re: [I] Support namespace in Pub/Sub commands [kvrocks]

Posted by "Chiro11 (via GitHub)" <gi...@apache.org>.

Chiro11 commented on issue #2097:
URL: https://github.com/apache/kvrocks/issues/2097#issuecomment-2033674167

   Do you still think this feat should be implemented? I think what @JxLi0921 said makes sense. But if you think this is valuable, please assign it to me. I am willing to have a try as my first issue.


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