hyperledger / fabric-x-committer / 29743468061

91%

main: 91%

Build Type

Pull #688

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` documentation and the affected tests.

- Resolves #682

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

Pull Request Pull Request #688: [loadgen] Remove the workload distribution

Coverage Stats

22 of 22 new or added lines in 4 files covered. (100.0%)

37 existing lines in 8 files now uncovered.

10331 of 11295 relevant lines covered (91.47%)

34871.6 hits per line