Add support of the command MOVEX

[Chiro11]

Close #1950
Some desc:

• MOVE key db is treated as dummy command, to be compatible with Redis.
• MOVEX key ns_token moves a key to new namespace.
  1. When namespace does not correspond to token, report an error.
  2. Do not support when requirepass is empty, report an error.
  3. When the source key does not exist or the target key already exists, return 0, otherwise, return 1.

[jihuayu]

Hi @Chiro11, thanks for your PR.
In the issues, we think we need to add a new command. Can you move the code to a new command?
#1950 (comment)

[Chiro11]

Hi @Chiro11, thanks for your PR. In the issues, we think we need to add a new command. Can you move the code to a new command? #1950 (comment)

You mean to use a new name? Do you have any idea?

[jihuayu]

Hi @Chiro11, thanks for your PR. In the issues, we think we need to add a new command. Can you move the code to a new command? #1950 (comment)

You mean to use a new name? Do you have any idea?

Yes. @git-hulk @PragmaTwice @mapleFU @enjoy-binbin Do you have some idea about new command name?

[git-hulk]

Hi @Chiro11, thanks for your PR. In the issues, we think we need to add a new command. Can you move the code to a new command? #1950 (comment)

You mean to use a new name? Do you have any idea?

Yes. @git-hulk @PragmaTwice @mapleFU @enjoy-binbin Do you have some idea about new command name?

Perhaps MOVEX is a good name for aligning with the CLUSTERX command.

[Chiro11]

Hi @Chiro11, thanks for your PR. In the issues, we think we need to add a new command. Can you move the code to a new command? #1950 (comment)

You mean to use a new name? Do you have any idea?

Yes. @git-hulk @PragmaTwice @mapleFU @enjoy-binbin Do you have some idea about new command name?

Perhaps MOVEX is a good name for aligning with the CLUSTERX command.

@jihuayu @git-hulk Modified as suggested.

[git-hulk]

@Chiro11 Thanks for your efforts and quick response.

[jihuayu]

Thank you for your code. @Chiro11
This PR contains a lot of refactoring unrelated to the movex command. Could you please move it to a new PR?

[git-hulk]

Rest are good to me, one comment inline.

[PragmaTwice]

MOVEX key ns_name ns_token

I think here the ns_name is useless.

[Chiro11]

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?

[Chiro11]

Thank you for your code. @Chiro11 This PR contains a lot of refactoring unrelated to the movex command. Could you please move it to a new PR?

@jihuayu I think it's basically ok. Just some typos and best practices.

[git-hulk]

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.

[Chiro11]

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.

[git-hulk]

@Chiro11 So let's remove it if you feel good about this.

[sonarcloud]

Quality Gate passed

Issues
7 New issues
0 Accepted issues

Measures
0 Security Hotspots
64.2% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud

[jihuayu]

Thanks @Chiro11. Good job! Can you add this command to our website?
https://kvrocks.apache.org/docs/supported-commands#key-commands

If you're willing, can you work for #2229?