Skip to content

Commit 155967c

Browse files
sgrebnovlukekim
authored andcommitted
Disable child metric registration under parent to prevent OOM (#10)
1 parent 0b20606 commit 155967c

1 file changed

Lines changed: 3 additions & 3 deletions

File tree

vortex-metrics/src/lib.rs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -89,9 +89,9 @@ impl VortexMetrics {
8989
/// Create a new metrics registry with additional tags. Metrics created in the
9090
/// child registry will be included in this registry's snapshots.
9191
pub fn child_with_tags(&self, additional_tags: impl Into<DefaultTags>) -> Self {
92-
let child = Self::new_with_tags(self.inner.default_tags.merge(&additional_tags.into()));
93-
self.inner.children.write().push(child.clone());
94-
child
92+
// TODO: Child metric registration under parent is temporarily disabled due to memory growth - children accumulate indefinitely.
93+
// https://github.com/vortex-data/vortex/discussions/5946
94+
Self::new_with_tags(self.inner.default_tags.merge(&additional_tags.into()))
9595
}
9696

9797
/// Returns the counter with the specified ID, creating a default instance if absent.

0 commit comments

Comments
 (0)