Skip to content

Commit 0a1e2d5

Browse files
emmaling27Convex, Inc.
authored andcommitted
Revert "Break down file storage usage gauge by component" (#30163)
Reverts get-convex/convex#30077 GitOrigin-RevId: d31e5fea10a3bf4e68d81bfacb54ab5387103ab2
1 parent 3a53718 commit 0a1e2d5

File tree

2 files changed

+1
-9
lines changed

2 files changed

+1
-9
lines changed

crates/events/src/testing.rs

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -176,10 +176,7 @@ impl UsageCounterState {
176176
},
177177
UsageEvent::CurrentVectorStorage { tables: _ } => todo!(),
178178
UsageEvent::CurrentDatabaseStorage { tables: _ } => todo!(),
179-
UsageEvent::CurrentFileStorage {
180-
total_size: _,
181-
by_component: _,
182-
} => todo!(),
179+
UsageEvent::CurrentFileStorage { total_size: _ } => todo!(),
183180
UsageEvent::CurrentDocumentCounts { tables: _ } => todo!(),
184181
}
185182
}

crates/events/src/usage.rs

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,3 @@
1-
use std::collections::BTreeMap;
2-
31
use async_trait::async_trait;
42
use serde::{
53
Deserialize,
@@ -110,10 +108,7 @@ pub enum UsageEvent {
110108
tables: Vec<TableDatabaseStorage>,
111109
},
112110
CurrentFileStorage {
113-
// TODO(ENG-7423): Clean up this old field once no databricks tables and queries rely on
114-
// it. They should use the by_component field instead.
115111
total_size: u64,
116-
by_component: BTreeMap<Option<String>, u64>,
117112
},
118113
CurrentDocumentCounts {
119114
tables: Vec<TableDocumentCount>,

0 commit comments

Comments
 (0)