From e2110601199dec4c2b44c04aaedde20a6ebf6138 Mon Sep 17 00:00:00 2001 From: aaltshuler Date: Sun, 5 Jul 2026 00:26:41 +0300 Subject: [PATCH] fix(engine): classify StagedTableNamespace opens as data-table opens Devin review follow-up on the opener unification: StagedTableNamespace's open_head opens a DATA table (its table_uri is the per-table physical path), so it belongs on the table_wrapper/data_open_count bucket, not the manifest one. Test-only impact (the namespace module is #[cfg(test)] since RFC-013 step 3a), but the classification should be right where the cost vocabulary is defined. --- crates/omnigraph/src/db/manifest/namespace.rs | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/crates/omnigraph/src/db/manifest/namespace.rs b/crates/omnigraph/src/db/manifest/namespace.rs index b3ab7522..9c622e75 100644 --- a/crates/omnigraph/src/db/manifest/namespace.rs +++ b/crates/omnigraph/src/db/manifest/namespace.rs @@ -107,11 +107,14 @@ impl StagedTableNamespace { } async fn open_head(&self) -> Result { + // A staged-table namespace opens a DATA table (its `table_uri` is the + // per-table physical path), so its opens count in the data-table + // bucket, not the internal/manifest one. crate::instrumentation::open_dataset( &self.table_uri(), crate::instrumentation::VersionResolution::Latest, None, - crate::instrumentation::manifest_wrapper(), + crate::instrumentation::table_wrapper(), ) .await }