cowsql / raft

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

v1: Trigger client callbacks after `raft_step()` #103

Closed freeekanayaka closed 11 months ago

freeekanayaka commented 11 months ago

Move the code that triggers client callbacks out of raft_step() into the legacy compatibility layer. In v1, managing this kind of logic will be responsibility of user code.

codecov[bot] commented 11 months ago

Codecov Report

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

Comparison is base (f0a5b2d) 74.85% compared to head (b92c209) 74.94%.

Files Patch % Lines
src/uv.c 80.00% 2 Missing and 6 partials :warning:
src/legacy.c 75.00% 4 Missing and 2 partials :warning:
src/replication.c 87.50% 2 Missing :warning:
src/fixture.c 66.66% 0 Missing and 1 partial :warning:
src/uv_truncate.c 0.00% 0 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #103 +/- ## ========================================== + Coverage 74.85% 74.94% +0.09% ========================================== Files 53 53 Lines 9357 9425 +68 Branches 2345 2354 +9 ========================================== + Hits 7004 7064 +60 - Misses 1014 1021 +7 - Partials 1339 1340 +1 ```

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