Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
105 changes: 0 additions & 105 deletions .github/workflows/validate-estimates.yml

This file was deleted.

3 changes: 1 addition & 2 deletions docs/performance.md
Original file line number Diff line number Diff line change
Expand Up @@ -96,9 +96,8 @@ No drift over the 3-minute window. p99 stayed well under 200 ms throughout.

2. **Pick `p99 ≤ 200 ms` as the SLO** at the recommended 27 RPS operating point. The full 3-minute sustained run met this.

3. **Re-baseline after issues #7, #45, or any worker-count change land.** Specifically:
3. **Re-baseline after issue #7 or any worker-count change lands.** Specifically:
- **#7 (UK NSPL, +1.79M postcodes)** — should not change per-request latency materially (still a dict lookup) but doubles in-memory state. Re-run to confirm.
- **#45 (happyGISCO outbound geocoding)** — would add a network call to the lookup path; the saturation RPS will drop sharply. **Mandatory** re-baseline.
- **Switching from single-worker to multi-worker** — likely the easiest large win. Each additional worker should approximately add another 30 RPS of headroom up to the container's CPU count.

4. **Don't run unattended high-concurrency tests.** Bombardier at c≥100 from a single source triggers platform-level connection refusal (`5xx`, dial timeouts) and risks short-term throttling. Keep scripted load below c=80.
Expand Down
Loading
Loading