Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Confusing error message while failed to parse the cluster slots #2401

Open
1 of 2 tasks
spihiker opened this issue Jul 11, 2024 · 3 comments
Open
1 of 2 tasks

Confusing error message while failed to parse the cluster slots #2401

spihiker opened this issue Jul 11, 2024 · 3 comments
Labels
enhancement type enhancement good first issue Good for newcomers help wanted Good for newcomers

Comments

@spihiker
Copy link

Search before asking

  • I had searched in the issues and found no similar issues.

Version

v2.9.0

Minimal reproduce step

CLUSTERX SETNODES "86f7e437faa5a7fce15d1ddcb9eaeaea377667b8 172.20.84.154 6666 master - 0-5460 3c363836cf4e16666669a25da280a1865c2d2874 172.20.84.154 6669 slave 86f7e437faa5a7fce15d1ddcb9eaeaea377667b8" 1
(error) ERR Slot is out of range

What did you expect to see?

normal as redis.

What did you see instead?

no

Anything Else?

no

Are you willing to submit a PR?

  • I'm willing to submit a PR!
@spihiker spihiker added the bug type bug label Jul 11, 2024
@PragmaTwice
Copy link
Member

PragmaTwice commented Jul 11, 2024

Could you try to add a \n?

CLUSTERX SETNODES "86f7e437faa5a7fce15d1ddcb9eaeaea377667b8 172.20.84.154 6666 master - 0-5460 \n  3c363836cf4e16666669a25da280a1865c2d2874 172.20.84.154 6669 slave 86f7e437faa5a7fce15d1ddcb9eaeaea377667b8" 1
@spihiker
Copy link
Author

After add a \n.It can run normally.Thank you.

@git-hulk git-hulk changed the title Slot is out of range Jul 12, 2024
@git-hulk
Copy link
Member

@PragmaTwice @spihiker This error message is very confusing for users, I have changed the issue title to track and improve it.

@git-hulk git-hulk added enhancement type enhancement and removed bug type bug labels Jul 12, 2024
@PragmaTwice PragmaTwice added help wanted Good for newcomers good first issue Good for newcomers labels Jul 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement type enhancement good first issue Good for newcomers help wanted Good for newcomers
3 participants