address feedback by Greg

anza-xyz · Jan 17, 2025 · 0e9c9ab · 0e9c9ab
1 parent 6df862a
commit 0e9c9ab
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/memory-management/src/jemalloc_monitor.rs b/memory-management/src/jemalloc_monitor.rs
@@ -120,6 +120,9 @@ impl MemPoolStats {
             .unwrap_or_else(|_| panic!("Prefix can not be over {} bytes long", NAME_LEN));
 
         self.data.push((key, Counters::default()));
+        // keep data sorted with longest prefixes first (this avoids short-circuiting)
+        // no need for this to be efficient since we do not run time in a tight loop and vec is typically short
+        self.data.sort_unstable_by(|a, b| b.0.len().cmp(&a.0.len()));
     }
 }
 

diff --git a/memory-management/src/jemalloc_monitor_metrics.rs b/memory-management/src/jemalloc_monitor_metrics.rs
@@ -84,7 +84,6 @@ pub fn setup_watch_memory_usage() {
         "solQuicTpu",
         "solQuicTpuFwd",
         "solRepairQuic",
-        "solGossipQuic",
         "solTurbineQuic",
     ] {
         mps.add(thread);