prisma-risk / tsoracle / 26373876980

95%

push

github

feat(consensus): unify HighWaterCommand advance naming across backends (#323)

Both consensus backends carried the same "advance the high-water to at
least N" command under divergent names — paxos `Advance { at_least }` vs
openraft `Bump { target }` — forcing a context switch when reading across
drivers.

Introduce a shared `AdvancePayload { at_least: u64 }` in
tsoracle-consensus that both drivers newtype-wrap as
`HighWaterCommand::Advance(AdvancePayload)`. Paxos keeps its Paxos-only
`Barrier` variant; openraft's `Bump`/`target` is renamed to
`Advance`/`at_least` and its `Display` updated to match. Each driver
re-exports `AdvancePayload` at its crate root so consumers can build
commands without depending on tsoracle-consensus directly.

The shared type's serde derives are gated behind the crate's existing
optional `serde` feature, which both drivers now enable.

This is wire-compatible: postcard encodes the newtype variant
`Advance(AdvancePayload { at_least })` byte-for-byte identically to the
former struct variants (variant tag followed by the `at_least` varint),
so no schema-version bump is needed. A new serialization test in
tsoracle-consensus pins `AdvancePayload { at_least: 5 }` to its bare
varint, and both per-backend on-disk frame fixtures continue to pin
their exact bytes.

Closes #267

59 of 59 new or added lines in 5 files covered. (100.0%)

11326 of 11927 relevant lines covered (94.96%)

476587.41 hits per line