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

feat(coordinator): support update service info when config changed #561

Merged
merged 18 commits into from
Nov 7, 2024

Conversation

mattisonchao
Copy link
Member

@mattisonchao mattisonchao commented Nov 7, 2024

Motivation

In practice, we might want to update the service public address for the existing cluster for some reason. This PR supports the functionalities by identifying the internal address as a logical node ID and updating the public address when the new term is used.

also, we can introduce a node Id that can help us update the internal address or other information introduced in the future.

Modification

  • Refresh service ensemble information at the newTerm safe point.
  • Use internalAddress as the logical node id.
  • Introduce a server indexes for better performance.

@mattisonchao mattisonchao force-pushed the feat.service.info.update.c branch from 1d25c6e to 6697e6f Compare November 7, 2024 12:48
@mattisonchao mattisonchao changed the title feat(coordinator): support update service info when new term feat(coordinator): support update service info when config changed Nov 7, 2024
@merlimat merlimat merged commit fb90db8 into main Nov 7, 2024
7 checks passed
@merlimat merlimat deleted the feat.service.info.update.c branch November 7, 2024 22:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants