Skip to content

feat(asap-tools): run ClickHouse E2E with frozen plans - #860

Merged
milindsrivastava1997 merged 3 commits into
mainfrom
859-featasap-tools-let-clickhouse-e2e-run-a-frozen-planner-output
Oct 13, 2026
Merged

milindsrivastava1997 merged 3 commits into
mainfrom
859-featasap-tools-let-clickhouse-e2e-run-a-frozen-planner-output

Conversation

@milindsrivastava1997

@milindsrivastava1997 milindsrivastava1997 commented Oct 13, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Support controller.prebuilt_output_dir in ClickHouse sketchdb runs.
  • Validate and deploy frozen SQL planner output without starting asap-planner.
  • Document separate SQL MILP plan generation and cover planner-selection paths.

Tests

  • PYTHONPATH=. python3 -m unittest discover -s tests (64 tests)
  • Pre-commit hooks

@milindsrivastava1997
milindsrivastava1997 merged commit 614184a into main Oct 13, 2026
12 checks passed
@milindsrivastava1997
milindsrivastava1997 deleted the 859-featasap-tools-let-clickhouse-e2e-run-a-frozen-planner-output branch October 13, 2026 16:21
zzylol added a commit that referenced this pull request Oct 13, 2026
Brings in #828 (the base, merged with later review changes), #860 and the
rest of main. Planner files, experiment_utils/config.py and
test_prebuilt_controller_output.py take main's versions (this branch never
changed them). experiment_run_e2e.py takes main's
copy_prebuilt_controller_output call. main_prometheus_client.py keeps both
align_to_interval and main's simulated_start_time, refusing a group that
sets both.

Co-Authored-By: Claude Opus 5.5 <[email protected]>
Claude-Session: https://claude.ai/code/session_01AnJ7LocWxguh3q4QWkCYae
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.

feat(asap-tools): let ClickHouse E2E run a frozen planner output

1 participant