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: distributed via raft #9

Closed
wants to merge 14 commits into from
Closed

feat: distributed via raft #9

wants to merge 14 commits into from

Conversation

jdockerty
Copy link
Owner

@jdockerty jdockerty commented Mar 29, 2024

TODO

  • Place behind a feature of distributed or raft so the functionality is split up nicely.
  • impl RaftNetwork/RaftNetworkFactory

Notes

raft-rs vs openraft

I initially started with raft-rs, but have found the documentation to be quite lacking and difficult to understand how to use the library itself. It's obvious that it only provides the consensus module of Raft and the user of the library is expected to provide the storage and transport layers; however, a decent example on how to do this is not present and using tikv as a guide felt impractical to me after awhile or trawling through the code base.

In search for an alternative I've found openraft, which also has a lot of community activity and the documentation goes through a full setup which I'm now looking to implement in this PR.

@jdockerty jdockerty self-assigned this Mar 29, 2024
jdockerty and others added 13 commits March 30, 2024 13:55
Tests will fail for now because the raft_tx causes a panic when not in use, if this is commented out we get them to pass again.

TODO: when raft is not in use, tests should pass still
Consider how we can run the gRPC server to receive the raft messages
and also drive the raft together
This was referenced Apr 10, 2024
@jdockerty
Copy link
Owner Author

Closing for now.

This is superseded by some basic replication strategy and raft/more complicated distributed models can be visited at a later date.

@jdockerty jdockerty closed this Apr 15, 2024
@jdockerty jdockerty deleted the feat/single-node-raft branch April 17, 2024 19:56
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