Skip to content
This repository has been archived by the owner on Feb 6, 2024. It is now read-only.

feat: add transfer leader procedure #67

Merged
merged 32 commits into from
Oct 14, 2022

Conversation

ZuLiangWang
Copy link
Contributor

@ZuLiangWang ZuLiangWang commented Sep 30, 2022

Which issue does this PR close?

#66

Rationale for this change

As mentioned in issue, add a procedure implementation

What changes are included in this PR?

Add TransferLeaderProcedure implementation

Are there any user-facing changes?

None.

How does this change test

Pass unit test.

@ZuLiangWang ZuLiangWang force-pushed the transfer_leader_procedure branch 3 times, most recently from 2a0a15d to e7a0593 Compare October 13, 2022 12:05
@ZuLiangWang ZuLiangWang force-pushed the transfer_leader_procedure branch 3 times, most recently from 02e8288 to ac0521d Compare October 13, 2022 12:34
server/coordinator/procedure/util.go Outdated Show resolved Hide resolved
server/coordinator/procedure/util.go Outdated Show resolved Hide resolved
server/coordinator/procedure/transfer_leader.go Outdated Show resolved Hide resolved
Copy link
Member

@ShiKaiWi ShiKaiWi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ShiKaiWi ShiKaiWi merged commit 1564d8a into apache:main Oct 14, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants