Chiro11 commented on PR #2225: URL: https://github.com/apache/kvrocks/pull/2225#issuecomment-2041436827
> > > `MOVEX key ns_name ns_token` > > > I think here the `ns_name` is useless. > > > > > > Theoretically it is. But I think this is more clear. @git-hulk @jihuayu What are the opinions of you two? > > The point of @PragmaTwice is we can find the `ns` by the token, so it's unnecessary to pass the namespace in the command arguments. I know. But I think `MOVEX key token` is a bit strange. I think it's reasonable to have a clear namespace. But I'm actually okay with both. Depends on you. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
