Skip to content

docs: ASAPPlanner and ASAPQuery-backend integration architecture - #512

Merged
zzylol merged 8 commits into
mainfrom
docs/asapplanner-system-integration
Sep 8, 2026
Merged

docs: ASAPPlanner and ASAPQuery-backend integration architecture#512
zzylol merged 8 commits into
mainfrom
docs/asapplanner-system-integration

Conversation

@zzylol

@zzylol zzylol commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Summary

Before / after

Before: architecture and migration ownership were implicit across the Planner pin, controller and serving paths.

After: an explicit shared semantic contract, system workflow, staged migration plan and implementation/acceptance tracker distinguish supported-profile implementation from review, merge and production rollout.

Scope

Documentation only; no runtime changes or repository rename in this PR. The runtime changes are separate sequential PRs and are not merged automatically.

Validation

  • git diff --check.
  • Baseline inspection: backend 95131d83972bb7a07d338e2a5af925a20c15ddce, historical Planner pin cb50219c582d43f53ab77d3a595bd1ea4a9aa119.
  • Runtime test results in the tracker were run on the combined implementation stack, not on this documentation branch.

@zzylol

zzylol commented Sep 8, 2026

Copy link
Copy Markdown
Contributor Author

Review and pushed fixes — 2026-09-08

Reviewed the 15 currently open PRs and their integration boundaries. This is self-review, not independent approval. No PR was merged; all code updates used normal commits/merges on existing PR branches.

PR Review outcome
#505 Fixed obsolete Planner API/pin, typed count/value updates, series/materialization identity, and multi-series TopK acceptance. Now based on #522.
#506 Updated to current #505; clarified that the negative matrix tests unregistered-query fallback, not missing arithmetic support. Process tests pass.
#509 Updated to current #505; compare all series at each range timestamp, including changing TopK winners. Process tests pass.
#511 Fixed legacy emitter dropping typed count updates; migrated both heap acceptance tests to registered temporal QueryPlans. Four canonical process tests pass.
#512 Updated merged Planner dependency and acceptance limits in the migration document.
#513 Reviewed compatible producer deduplication and conflicting-contract rejection; no additional finding in this pass. Covered by combined compiler/process tests.
#514 Reviewed merged-API adaptation and fail-closed unsupported operations; no additional finding in this pass.
#515 Fixed ranking that dropped duplicate algorithm candidates and could panic; pinned merged Planner #356 revision 378a754.
#516 Reviewed packed SUM/count, arithmetic grouping/source restrictions and non-additive reduction fallback; inherited #515 fixes.
#517 Reviewed publication, frame identities and failed-generation isolation; inherited fixes; distributed process test passes.
#518 Reviewed startup/live workload selection and query-scoped certificates; inherited fixes.
#519 Reviewed shared lifecycle demand and conflicting evidence rejection; inherited fixes.
#520 Reviewed actual Collector Rust update/output counters; inherited fixes; distributed process test passes.
#521 Reviewed staging cleanup/retry and concurrent same-semantic cutover; inherited fixes; distributed process test passes.
#522 Fixed quote preparation incorrectly requiring the default warm plan to compile; available exact manifests now survive unavailable warm implementations. Regression and all 29 binary tests pass.

Validation

#511 acceptance update

User confirmed migration to the new contract. Commit 1a85748 replaces the two no-QueryPlan heap TopK tests with production backend-local tests of registered topk(3, count_over_time(top_endpoint_qps[5s])).

CMS-with-heap and CountSketch-with-heap each return all three expected identities/counts (gamma=200, zeta=150, alpha=100), with timestamps, two range steps and Remote Write retry deduplication checked. Raw values are 17 to distinguish counts from weighted sums. Unregistered instantaneous TopK follows the configured exact fallback.

All four canonical compatibility process tests passed; the remaining legacy test file compiles. This resolves the two-test contract-migration decision above, not all unrelated legacy OTLP serving acceptance. No independent approval or merge performed.

# Conflicts:
#	docs/design_docs/README.md
@zzylol
zzylol merged commit d4b196b into main Sep 8, 2026
1 check passed
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