Skip to content

Commit 230f6b6

Browse files
committed
history: minor tweaks to TestThrashing
1 parent 1667144 commit 230f6b6

File tree

3 files changed

+14
-14
lines changed

3 files changed

+14
-14
lines changed

pkg/kv/kvserver/asim/history/testdata/TestThrashing/initial_outlier

Lines changed: 11 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -3,16 +3,16 @@ echo
33
An initial outlier leads to a large normalization factor,
44
i.e. low thrashing percentage. This isn't necessarily good, if this becomes
55
an issue we could use an inter-quantile range instead.
6-
input: [10250 5004 12 1 2]
6+
input: [10250 13 12 1 2]
77
10250 ┼╮
8-
9225 ┤╰
9-
8200 ┤ ╰─
10-
7175 ┤ ╰─
11-
6150 ┤ ╰─
12-
5126 ┤ ╰─
13-
4101 ┤ ╰─
14-
3076 ┤ ╰─
15-
2051 ┤ ╰─
16-
1026 ┤ ╰─
17-
1 ┤ ────────────────────
8+
9225 ┤╰╮
9+
8200 ┤
10+
7175 ┤
11+
6150 ┤
12+
5126 ┤
13+
4101 ┤
14+
3076 ┤
15+
2051 ┤
16+
1026 ┤
17+
1 ┤ ╰─────────────────────────────
1818
tdtv=0.07% (7.3/10249.0) uptv=1.0 dntv=10249.0 runs=2

pkg/kv/kvserver/asim/history/testdata/TestThrashing/negative_crossover

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
echo
22
----
33
Oscillating across zero in both directions, mostly to check that
4-
negative numbers don't cause issues.'
4+
negative numbers don't cause issues.
55
input: [-10 -4 -6 -1 4 5 -2 6]
66
6.00 ┤ ╭
77
4.40 ┤ ╭─────╮ ╭╯

pkg/kv/kvserver/asim/history/trashing_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ func TestThrashing(t *testing.T) {
7676
desc: `An initial outlier leads to a large normalization factor,
7777
i.e. low thrashing percentage. This isn't necessarily good, if this becomes
7878
an issue we could use an inter-quantile range instead.`,
79-
vs: []float64{10250, 5004, 12, 1, 2},
79+
vs: []float64{10250, 13, 12, 1, 2},
8080
},
8181
{
8282
name: "final_outlier",
@@ -94,7 +94,7 @@ beginning. The trend-aware thrashing measure (tdtv) should be small.
9494
{
9595
name: "negative_crossover",
9696
desc: `Oscillating across zero in both directions, mostly to check that
97-
negative numbers don't cause issues.'`,
97+
negative numbers don't cause issues.`,
9898
vs: []float64{-10, -4, -6, -1, 4, 5, -2, 6},
9999
},
100100
} {

0 commit comments

Comments
 (0)