[zzylol]

Why

The Collector carries code that no build, test, deploy script or CI job uses, and its CollectorPlan contract has two fields that can only take one value. That code costs review and build time, and the "pane" wording makes readers think the Collector runs something other than a tumbling window. The Collector runs exactly one window type: a tumbling window of width W.

What

• Remove the sliding-window mode from asap-precompute-go. That covers AggregationMode.Sliding, WindowSpec.Slide, the ring of closed windows, slideMs, panesPerWindow, rotateSlidingLocked and their tests. Without the sliding mode, rotationStepMs only forwarded to windowSizeMs, so its callers now call windowSizeMs directly.
• CollectorPlan contract change, with no compatibility alias:
  • remove pane_secs and state_layout (they had to equal window_secs and "anchored-pane-v1");
  • rename window_secs to tumbling_window_secs (Go: TumblingWindowSecs).
    The Go decoder, the Rust mirror, the shared fixture and all tests are updated. window_implementation_id stays collector-tumbling-v1.
• Delete asap-gorilla-rust/, the Serf processor/exporter/receiver, otel_collector_benchmark/, and the unused PipelineConfig loader in the Go SDK patch.
• Delete the unused S3-direct cold settings from asap_edge and remove them from the one deploy config that still set them.
• Remove about 25 comment citations of design-gos-unified-edge-telemetry.md, which was deleted when the design docs were consolidated. Fix other stale file paths and status lines.
• Where a doc means the Collector's window, it now says "tumbling window" instead of "pane".

How

Deleted	Why it was dead	Evidence
Sliding mode, WindowSpec.Slide, closed-window ring, slideMs, panesPerWindow, rotateSlidingLocked, windowState.sketchFactory, rotationStepMs	The runtime builds configs only from CollectorPlan, and the decoder rejects every framework except tumbling. asap_edge also checks Mode == Tumbling.	git grep -w Sliding: no users outside window.go, config.go and their tests. The only Slide reader was the asap_edge shape check, which compared it to the window size.
pane_secs, state_layout	Validation forced pane_secs == window_secs and state_layout == "anchored-pane-v1", so neither field carried information.	collector_plan.go / collector_plan.rs validation. No other repo produces CollectorPlan on main.
asap-gorilla-rust/	No Cargo or Go dependency in this repo or in ASAPQuery-backend main.	run_demo.sh only checked that the directory existed and passed it as a --build-context. Backend main has no data_plane/Dockerfile or control_plane/Dockerfile and no reference to gorilla.
serfprocessor, serfexporter, serfreceiver	No deploy config uses the serf or serfhttp component types. The MVP README excludes the Serf comparison.	git grep -i serf: only the builder manifest and comments.
otel_collector_benchmark/	Not built, run or referenced.	The only hit was a .gitignore line, also removed.
opentelemetry-go-patch/sdk/metric/pipelineconfig.go (+ test)	A file the patch adds (not in upstream opentelemetry-go). Only its own test calls it.	git grep PipelineConfig: self-references only.
asap_edge cold endpoint, tsdb_bucket, tenant, region, access_key_id, secret_access_key, use_ssl	Only config validation read them. No code writes to S3 from the edge.	grep in the processor: no reads outside config_validate.go. Removed from asap-otel-agent-asap-gzip.yaml, the only config that set them. ship_endpoint is untouched.

run_demo.sh no longer requires or passes asap-gorilla-rust. The asap-otel builder manifest no longer lists the three Serf components.

Before

"window_secs": 60,
"abstract_window_framework": "tumbling",
"window_implementation_id": "collector-tumbling-v1",
"pane_secs": 60,
"state_layout": "anchored-pane-v1",

WindowSpec{Size, Slide, AllowedLateness}, with a Sliding mode that no producer could configure.

After

"tumbling_window_secs": 60,
"abstract_window_framework": "tumbling",
"window_implementation_id": "collector-tumbling-v1",

WindowSpec{Size, AllowedLateness}, with Tumbling and Batch modes. A plan that still sends window_secs, pane_secs or state_layout is now rejected as having unknown fields (the Rust decoder uses deny_unknown_fields, and the Go decoder rejects unknown fields too). This is the intended contract change.

Verification

• cd asap-precompute-go && go vet ./... && go test ./...: pass.
• cd opentelemetry-collector-contrib-patch/processor/asapedgeprocessor && go vet ./... && go test ./...: pass.
• opentelemetry-go/sdk/metric (after restore): go build ./... && go vet . && go test . && go test ./internal/aggregate -run 'TestRowSampledSketch_': pass.
• cd asap-precompute-rs && cargo fmt -- --check && cargo clippy --all-features --all-targets && cargo test --all-features: pass.
• python3 -m unittest discover -s deploy/mvp-multinode/scripts/tests -p 'test_*.py': 20 tests OK.
• bash -n deploy/mvp-multinode/scripts/run_demo.sh: OK.
• The asap-otel distribution builds with OCB v0.141.0 from the updated manifest. asap-otel components lists no Serf component.
• CI's MVP collector check fails on this PR as it already does on main: CI checks out asap_sketchlib main, and asap-precompute-rs does not build against its newer DdSketch API. This PR does not change that crate's sketch code; moving the collector to the latest asap_sketchlib and sketchlib-go is separate work.