-
Notifications
You must be signed in to change notification settings - Fork 2
Clarify/implement scope (currently always empty) #235
Copy link
Copy link
Open
Labels
area/apiHTTP handlers, routing, middlewareHTTP handlers, routing, middlewarearea/ingestIngest pipeline (Bento, batching, DLQ)Ingest pipeline (Bento, batching, DLQ)documentationImprovements or additions to documentationImprovements or additions to documentationenhancementNew feature or requestNew feature or request
Description
Activity
Metadata
Metadata
Assignees
Labels
area/apiHTTP handlers, routing, middlewareHTTP handlers, routing, middlewarearea/ingestIngest pipeline (Bento, batching, DLQ)Ingest pipeline (Bento, batching, DLQ)documentationImprovements or additions to documentationImprovements or additions to documentationenhancementNew feature or requestNew feature or request
Type
Projects
- StatusShow more project fieldsBacklog
Area: ingest — confusion / FYI · found via WaveHouse-Stats dogfooding
Expected: know whether
scope(inEventMessage) provides any isolation today.Actual: it's always empty — the multi-tenant scope concept exists in the wire format but isn't populated or used (
internal/api/ingest.go:150-151, with matching// TODO: impl scopeinstructured_query.go:121andstream.go:54).Impact (Stats): none now (single-tenant); flagging so nobody assumes
scopeisolates anything if tenants are added.Scope: document that
scopeis currently inert, and/or implement population as part of multi-tenant work.Related: per-table policy storage + per-table authorization scopes #214 (the policy side of multi-tenant;
scopeis the runtime-population half).From
WAVEHOUSE-FEEDBACK.md(WaveHouse-Stats dogfooding); validated by code-read against0f8826con 2026-06-04.