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.

Sign up for GitHub

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

Jump to bottom

When memberChange(), memberAddressInfos should record Up State memberAddressInfos #4450

Closed
horizonzy opened this issue Dec 10, 2020 · 1 comment
Closed

When memberChange(), memberAddressInfos should record Up State memberAddressInfos #4450

horizonzy opened this issue Dec 10, 2020 · 1 comment
Labels
kind/bug Category issues or prs related to bug.
Milestone
1.4.1

Comments

@horizonzy
Copy link
Collaborator

Describe the bug
com.alibaba.nacos.core.cluster.ServerMemberManager#memberChange
Now the logic will record param members, whatever the member's state.
@horizonzy horizonzy mentioned this issue Dec 10, 2020
Merged
@KomachiSion
Copy link
Collaborator

Please discuss with @chuntaojun first, because he is modifying this part codes in #4371.

@KomachiSion KomachiSion added kind/discussion Category issues related to discussion kind/bug Category issues or prs related to bug. and removed kind/discussion Category issues related to discussion labels Dec 11, 2020
@KomachiSion KomachiSion added this to the 1.4.1 milestone Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
kind/bug Category issues or prs related to bug.
Projects
None yet
Milestone
1.4.1
Development

No branches or pull requests
2 participants
@horizonzy @KomachiSion