hyperledger / fabric-x-committer
91%
main: 91%

LAST BUILD BRANCH: loadgen-probability-validation
DEFAULT BRANCH: main
Repo Added
Files 127
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH remove-loadgen-distribution
branch: remove-loadgen-distribution
CHANGE BRANCH
x

coverage: 91.405% (-0.1%) from 91.509%
29756083315

Pull #688

github

liran-funaro 
[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>
Pull Request #688: [loadgen] Remove the workload distribution

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

13 existing lines in 6 files now uncovered.

10305 of 11274 relevant lines covered (91.41%)

35205.93 hits per line

Relevant lines Covered
11274 RELEVANT LINES 10305 COVERED LINES
35205.93 HITS PER LINE
Source Files on remove-loadgen-distribution

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
29756083315 remove-loadgen-distribution [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... Pull #688 liran-funaro github
91.41
29754062057 remove-loadgen-distribution [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... Pull #688 liran-funaro github
91.48
29749692684 remove-loadgen-distribution [loadgen] Remove the workload distribution #### Type of change - Improvement #### Description - The load generator's `Distribution` descriptor supported five variants (`const`, `uniform`, `normal`, `bernoulli`, `discrete`), but only `const` wa... Pull #688 liran-funaro github
91.45
29743468061 remove-loadgen-distribution [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... Pull #688 liran-funaro github
91.47
See All Builds (864)