Closed CrystalAnalyst closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 75.72%. Comparing base (
e35b35a
) to head (97aa29d
). Report is 121 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Please briefly answer these questions:
what problem are you trying to solve? (or if there's no problem, what's the motivation for this change?) Update testing logic to keep aligned with early return in fast path. Since we can return early in fast path when the number of conlicts is so large that we cannot reach SuperQuorum. So we DO NOT NEED to force all the 5 servers to handle the propose. related issue : #840
what changes does this pull request make? Remove the counter, and the rest part works fine (Successfully fallback to slow path) .
are there any non-obvious implications of these changes? (does it break compatibility with previous versions, etc)