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

Fix wrong replica's port number(=0) if the listening port is not set #1763

Merged

Conversation

xiaofan8421
Copy link
Contributor

Background:
For some reason, I used Go to implement my slaves based on kvrocks. But by redis-cli to get replication-info from kvrocks(no matter what master/slave), the peer port is 0 which is wrong.
Reproduce:
Not only localhost but also the cross-machine can reproduce the bug scene.

So, I fixed it. Maybe it is not your code standard, Please review it.

before:
img_v2_058c11fb-db25-4852-acc0-d3f0d757749h
after:
1b30b756-37a1-4846-a9d7-b1d585a53bf0

@git-hulk
Copy link
Member

@xiaofan8421 Thanks for your good catch! Leave a comment inline.

@git-hulk git-hulk changed the title bugFix: fix info replication about slaves conn port. Fix wrong replica's port number(=0) if the listening port is not set Sep 18, 2023
@torwig torwig force-pushed the bugfix/roy/info-slaves-peer-addr branch from 1d5f4d9 to 3dab859 Compare September 18, 2023 08:58
@PragmaTwice
Copy link
Member

@xiaofan8421 Hi, would you like to finish the final modification according to @git-hulk before the 2.6.0 cherry-picking period? (refer to #1767)

If it can be merged before the deadline, it can be shipped to kvrocks 2.6.0 smoothly : )

If you are busy, we can help you to do that.

@xiaofan8421 xiaofan8421 force-pushed the bugfix/roy/info-slaves-peer-addr branch from 640ca10 to b7df06e Compare September 20, 2023 06:40
@xiaofan8421
Copy link
Contributor Author

@xiaofan8421 Hi, would you like to finish the final modification according to @git-hulk before the 2.6.0 cherry-picking period? (refer to #1767)

If it can be merged before the deadline, it can be shipped to kvrocks 2.6.0 smoothly : )

If you are busy, we can help you to do that.

Thanks for your mention. Update PR. Please review it.

@git-hulk git-hulk merged commit 888265d into apache:unstable Sep 21, 2023
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants