cowsql / raft

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

convert: Make initial blank no-op entry optional #124

Closed freeekanayaka closed 10 months ago

freeekanayaka commented 10 months ago

Don't force leaders to create the initial no-op entry at the beginning of their term, and let the application decide.

codecov[bot] commented 10 months ago

Codecov Report

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

Comparison is base (418d6d3) 73.22% compared to head (41a8182) 73.52%.

Files Patch % Lines
src/replication.c 60.00% 0 Missing and 6 partials :warning:
src/client.c 58.33% 3 Missing and 2 partials :warning:
src/election.c 40.00% 0 Missing and 3 partials :warning:
src/raft.c 85.00% 1 Missing and 2 partials :warning:
src/convert.c 75.00% 0 Missing and 1 partial :warning:
src/tick.c 87.50% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #124 +/- ## ========================================== + Coverage 73.22% 73.52% +0.29% ========================================== Files 52 52 Lines 9614 9647 +33 Branches 2345 2354 +9 ========================================== + Hits 7040 7093 +53 + Misses 1260 1232 -28 - Partials 1314 1322 +8 ```

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