Skip to content

Commit 11c2c53

Browse files
authored
chore: size the 0.156.0 release memory exception to the measured regressions (#1086)
1 parent d9d3477 commit 11c2c53

3 files changed

Lines changed: 13 additions & 12 deletions

File tree

‎scripts/release_performance/constants.py‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -86,23 +86,23 @@
8686
name="lineage hub downstream",
8787
project=INSPECTION_PROJECT,
8888
sqb_args=("lineage", SHARED_DIAMOND_HUB, "--direction", "downstream"),
89+
max_rss_ratio=1.75,
8990
),
9091
BenchmarkCommand(
9192
name="lineage rollup upstream",
9293
project=INSPECTION_PROJECT,
9394
sqb_args=("lineage", SHARED_DIAMOND_ROLLUP, "--direction", "upstream"),
95+
max_rss_ratio=1.75,
9496
),
9597
BenchmarkCommand(
9698
name="lineage column trace",
9799
project=INSPECTION_PROJECT,
98100
sqb_args=("lineage", f"{SHARED_DIAMOND_ROLLUP}.amount", "--direction", "upstream"),
99-
max_rss_ratio=2.0,
100101
),
101102
BenchmarkCommand(
102103
name="dag --json",
103104
project=INSPECTION_PROJECT,
104105
sqb_args=("dag", "--json"),
105-
max_rss_ratio=1.5,
106106
),
107107
BenchmarkCommand(
108108
name="scope --json",
@@ -118,7 +118,6 @@
118118
),
119119
BenchmarkCommand(
120120
name="compile (one-model edit)",
121-
max_rss_ratio=1.35,
122121
project=INSPECTION_PROJECT,
123122
sqb_args=("compile", "--json"),
124123
edits_model=True,
@@ -128,6 +127,7 @@
128127
name="dense compile (warm cache)",
129128
project=DENSE_PROJECT,
130129
sqb_args=("compile", "--json"),
130+
max_rss_ratio=1.45,
131131
max_time_ratio=MAX_COMPILE_TIME_RATIO,
132132
),
133133
BenchmarkCommand(

‎tests/e2e/scripts/release_performance/test_compare_release_performance.py‎

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -39,9 +39,10 @@
3939
"| `plan --json` |",
4040
"| `build (empty warehouse)` |",
4141
"| `lineage column trace` |",
42-
"Peak RSS limit for `lineage column trace`: 2.00x "
43-
"(+100%; temporary memory exception).",
44-
"Peak RSS limit for `dag --json`: 1.50x (+50%; temporary memory exception).",
42+
"Peak RSS limit for `lineage hub downstream`: 1.75x "
43+
"(+75%; temporary memory exception).",
44+
"Peak RSS limit for `dense compile (warm cache)`: 1.45x "
45+
"(+45%; temporary memory exception).",
4546
"Benchmark projects are generated per side",
4647
"**Result: ",
4748
),
@@ -121,9 +122,9 @@ def test_given_two_installations_when_comparing_release_performance_then_reports
121122
]
122123
for command in recorded["commands"]:
123124
assert command["max_rss_ratio"] == {
124-
"lineage column trace": 2.0,
125-
"dag --json": 1.5,
126-
"compile (one-model edit)": 1.35,
125+
"lineage hub downstream": 1.75,
126+
"lineage rollup upstream": 1.75,
127+
"dense compile (warm cache)": 1.45,
127128
}.get(command["name"])
128129

129130

‎tests/unit/scripts/release_performance/_helpers/test_verdict.py‎

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -290,9 +290,9 @@ def test_given_configured_command_when_judging_time_then_its_own_limit_applies(
290290
description=f"{command.name} enforces its RSS allowance and time limits",
291291
command=command,
292292
expected_rss_ratio={
293-
"lineage column trace": 2.0,
294-
"dag --json": 1.5,
295-
"compile (one-model edit)": 1.35,
293+
"lineage hub downstream": 1.75,
294+
"lineage rollup upstream": 1.75,
295+
"dense compile (warm cache)": 1.45,
296296
}.get(command.name, 1.25),
297297
)
298298
for command in BENCHMARK_COMMANDS

0 commit comments

Comments
 (0)