Skip to content

perf: remove redundant env copy - #81

Merged
tonyd33 merged 3 commits into
release/0.3from
perf/remove-redundant-env-copy
Aug 31, 2026
Merged

perf: remove redundant env copy#81
tonyd33 merged 3 commits into
release/0.3from
perf/remove-redundant-env-copy

Conversation

@tonyd33

@tonyd33 tonyd33 commented Aug 31, 2026

Copy link
Copy Markdown
Owner

No description provided.

@github-actions

github-actions Bot commented Aug 31, 2026

Copy link
Copy Markdown

Benchmark passed

Query time (limit 35s, regression threshold 10%).
6 machines × 3 iterations per side; head and base run interleaved on the same machine.
Δ is the paired per-machine estimate, so cross-machine variance cancels.
Base: 21090fb

Stage Head median Base median Δ (paired) 95% confidence
Read 0.84s 0.83s -0.2% head is no more than 2.7% slower
Parse 130.46s 130.38s -1.2% head is at least 0.4% faster
Query 31.47s 33.98s -9.1% head is at least 5.5% faster
Per-machine query delta
Machine Δ
1 -4.1%
2 -5.3%
3 -12.1%
4 -6.0%
5 -13.7%
6 -13.0%
Confidence curve

Paired across 6 machines (df=5); sample size is machines, not runs, since same-machine runs are correlated.

  • 95% confident: head is at least 5.5% faster
  • 99% confident: head is at least 3.0% faster
Head faster by at least Confidence
-25% 100.0%
-20% 100.0%
-15% 100.0%
-10% 100.0%
-5% 100.0%
-2% 99.9%
+0% 99.8%
+2% 99.4%
+5% 96.5%
+10% 32.0%
+15% 0.9%
+20% 0.1%
+25% 0.0%
+30% 0.0%

Negative rows read as regressions: the +0% row is the confidence that head is faster at all.

@tonyd33
tonyd33 force-pushed the perf/remove-redundant-env-copy branch from fbb2112 to 16af5b3 Compare August 31, 2026 02:11
@tonyd33
tonyd33 force-pushed the perf/remove-redundant-env-copy branch from 16af5b3 to 3011183 Compare August 31, 2026 02:25
@tonyd33
tonyd33 merged commit f60cf7a into release/0.3 Aug 31, 2026
11 checks passed
@tonyd33
tonyd33 deleted the perf/remove-redundant-env-copy branch August 31, 2026 02:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant