liran-funaro / fabric-x-committer / 29754058440

91%

main: 92%

Build Type

push

github

Committed by liran-funaro

Commit Message

[loadgen] Remove the workload distribution

- Improvement

- The load generator's `Distribution` descriptor supported five variants
(`const`, `uniform`, `normal`, `bernoulli`, `discrete`), but only `const`
was ever used. Load generation holds each parameter at a fixed value so
results are deterministic and comparable across runs, so the unused
variants only added config surface, sample-YAML documentation, generator
types, and tests with no realized benefit. This collapses `Distribution`
down to plain constant values.

- Delete `distributions.go` and the orphaned
`Const`/`FloatToInt`/`FloatToPositiveInt`/`FloatToBoolean` generators;
`MultiGenerator.Count` is now a plain `int`.
- Replace the `*Distribution` profile fields with plain scalars:
read/read-write/blind-write counts and query size become `uint32`, the
minimal invalid-keys portion becomes `Probability`, and the dependency
gap becomes `uint64` (min 1).
- Keep per-transaction conflict injection (invalid signatures and
dependency probability) via a small `bernoulliGenerator`
(`rnd.Float64() < p`) with identical semantics to the previous
Bernoulli-based path.
- Update the sample `loadgen.yaml` and the `loadgen_shared.yaml.tmpl`
config template to the plain-number syntax, plus the affected tests.

- Resolves #682

Signed-off-by: Liran Funaro <liran.funaro@gmail.com>

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>

Coverage Stats

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

52 existing lines in 13 files now uncovered.

10309 of 11274 relevant lines covered (91.44%)

35136.78 hits per line