search

sofastack / sofa-jraft

A production-grade java implementation of RAFT consensus algorithm.
https://www.sofastack.tech/projects/sofa-jraft/
Apache License 2.0
3.57k stars 1.14k forks source link

Modify the counter example to avoid misunderstanding #228

Closed fengjiachun closed 4 years ago

fengjiachun commented 5 years ago

Your question

In Counter demo, the get request is not readIndex read, the user will imitate the demo in his own implementation, who will be misled.

Your scenes

Your advice

Environment

sofastack-bot[bot] commented 5 years ago

Hi @fengjiachun, we detect non-English characters in the issue. This comment is an auto translation by @sofastack-robot to help other users to understand this issue.

We encourage you to describe your issue in English which is more friendly to other users.

Your question Counter demo, the get request is not linearly consistent, the user will write his own implementation according to the demo, it is easy to be misled ### Your scenes ### Your advice ### Environment - SOFAJRaft version: - JVM version (eg java -version): - OS version (eg uname -a): - Maven version: - IDE version:

masaimu commented 5 years ago

Maybe I can optimize it when i prepare the SOFAJRaft live.

killme2008 commented 4 years ago

@masaimu 这个可以做下。