baidu / braft

An industrial-grade C++ implementation of RAFT consensus algorithm based on brpc, widely used inside Baidu to build highly-available distributed systems.
Apache License 2.0
3.86k stars 865 forks source link

raft_protocol.md 中有一处描述的不准确? #378

Open longuan opened 1 year ago

longuan commented 1 year ago

在看Raft文档时,发现log-recovery那个章节中,对那个图片过程描述的不准确。

文档地址:raft_protocol.md

log_recovery.png 图片中 c到d2那个过程这样描述是不是更好点:S1没有crash,依然作为term 4的leader,将term4产生的log entry复制到大多数节点之后做commit,term 2的log entry也会顺带被commit。

如果按照现在描述,c到d2的过程,S1 crash之后,重新被S1、S2、S3选为Term5的Leader,term 2和term 4的log entries即使被复制到大多数节点也不会做commit。

PFZheng commented 1 year ago

这儿的描述是为了了d1保持一致