apunkt/omnigraph
1
0
Fork 0
mirror of https://github.com/ModernRelay/omnigraph.git synced 2026-07-12 03:12:11 +02:00
Code Issues Projects Releases Packages Wiki Activity Actions 2

fix(engine): classify StagedTableNamespace opens as data-table opens

Browse source 
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.
This commit is contained in:
aaltshuler 2026-07-05 00:26:41 +03:00 • committed by Andrew Altshuler
parent 451585ee59
commit e211060119
1 changed files with 4 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

5
crates/omnigraph/src/db/manifest/namespace.rs
View file

@ -107,11 +107,14 @@ impl StagedTableNamespace {
}
async fn open_head(&self) -> Result<Dataset> {
// 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
}