Problem. Per report 06 A1/L1/L2: NewCollector, SaveSnapshot/BulkInsertSnapshots, all Query*, and partition management have zero production callers; only RefreshMaterializedViews is wired, refreshing views over a table nothing writes to. Partitions stop being created after the seeded 3 months (M3); retention never runs. ~300 LOC + 4 mock files + an indexed FK column maintained for an unused API.
Evidence. Report 06 (A1, L1, L2, M3). Repo-wide grep confirms no live caller.
Suggested fix (explicit decision required, no-delete rule). Either (a) finish: wire the collector to an EventBridge schedule + monthly partition-maintenance job + expose Query* via the dashboard (then I-12's guards become mandatory), or (b) remove the unused surface. Do not leave the half-built scaffold.
References. Source: report 06 (A1, L1, L2, M3). Gated by I-12 (guards before wiring).
Filed from automated adversarial code review (see docs/code-review/). Source finding(s): 06-A1, 06-L1, 06-L2.
Problem. Per report 06 A1/L1/L2:
NewCollector,SaveSnapshot/BulkInsertSnapshots, allQuery*, and partition management have zero production callers; onlyRefreshMaterializedViewsis wired, refreshing views over a table nothing writes to. Partitions stop being created after the seeded 3 months (M3); retention never runs. ~300 LOC + 4 mock files + an indexed FK column maintained for an unused API.Evidence. Report 06 (A1, L1, L2, M3). Repo-wide grep confirms no live caller.
Suggested fix (explicit decision required, no-delete rule). Either (a) finish: wire the collector to an EventBridge schedule + monthly partition-maintenance job + expose Query* via the dashboard (then I-12's guards become mandatory), or (b) remove the unused surface. Do not leave the half-built scaffold.
References. Source: report 06 (A1, L1, L2, M3). Gated by I-12 (guards before wiring).
Filed from automated adversarial code review (see docs/code-review/). Source finding(s): 06-A1, 06-L1, 06-L2.