cowsql / raft

Asynchronous C implementation of the Raft consensus protocol
https://raft.readthedocs.io
Other
53 stars 6 forks source link

Apply committed entries eagerly on followers #95

Closed freeekanayaka closed 1 year ago

freeekanayaka commented 1 year ago

Don't make followers wait for entries to be persisted on disk before advancing their commit index.

codecov[bot] commented 1 year ago

Codecov Report

Attention: 9 lines in your changes are missing coverage. Please review.

Comparison is base (7896ffc) 75.00% compared to head (e790e42) 74.98%. Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #95 +/- ## ========================================== - Coverage 75.00% 74.98% -0.03% ========================================== Files 53 53 Lines 9319 9317 -2 Branches 2322 2323 +1 ========================================== - Hits 6990 6986 -4 Misses 1009 1009 - Partials 1320 1322 +2 ``` | [Files](https://app.codecov.io/gh/cowsql/raft/pull/95?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cowsql) | Coverage Δ | | |---|---|---| | [src/replication.c](https://app.codecov.io/gh/cowsql/raft/pull/95?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=cowsql#diff-c3JjL3JlcGxpY2F0aW9uLmM=) | `68.16% <40.00%> (-0.41%)` | :arrow_down: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.