Skip to content

Ensure frozen perf uses standard worker size - #1150

Merged
bill-ph merged 1 commit into
mainfrom
codex/perf-standard-worker
Sep 2, 2026
Merged

Ensure frozen perf uses standard worker size#1150
bill-ph merged 1 commit into
mainfrom
codex/perf-standard-worker

Conversation

@bill-ph

@bill-ph bill-ph commented Sep 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • pass the deployed standard Duckgres worker CPU and memory into the frozen perf scenario Job
  • request that exact worker shape through PGWire startup options so measured queries bypass the exploratory tier
  • document worker sizing defaults and add coverage for scenario wiring, Job environment forwarding, and the generated PGWire DSN

Why

Frozen perf connections did not specify a worker profile, so they entered the exploratory tier. Queries that spilled successfully never triggered memory-error escalation and were measured on the small worker instead of the standard worker. Explicit sizing makes the Duckgres and Trino execution-resource comparison stable.

Testing

  • just test-unit
  • just lint

@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown

Test Impact Plan

Deterministic summary of how this PR changes tests, CI runners, and coverage-risk signals.

Summary

Area Added Changed Deleted
Test files 0 9 0
E2E/journey files 0 0 0
Workflow files 0 0 0

Signals

  • Test cases: +0 / -0
  • Assertions: +5 / -0
  • Skips or known failures added: 0
  • Workflow continue-on-error added: 0
  • Workflow path filters added: 0
  • Test commands removed from justfile: 0
  • E2E/journey retry lines added: 0

Coverage risk: neutral or increased

No coverage-reduction warnings detected.

@bill-ph
bill-ph merged commit fa8cb4b into main Sep 2, 2026
30 checks passed
@bill-ph
bill-ph deleted the codex/perf-standard-worker branch September 2, 2026 21:49
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