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 peerid #455

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

fix peerid #455

wants to merge 1 commit into from

Conversation

CkTD
Copy link

@CkTD CkTD commented Jun 4, 2024

PeerId 增加了 role,但是它的 operator < 和 operator == 没有考虑 role。
PeerId 的 operator < 被 std::map 使用(包括 Configuration/node_manager)。没有考虑 role 会导新建一个 addr、idx 相同但是角色不同的节点时失败。

ehds added a commit to ehds/mbraft that referenced this pull request Jun 10, 2024
pick: baidu/braft#455

Co-authored-by: yangzining.yzn <[email protected]>
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.

1 participant