Remove metrics from accounts-db

nitro-svm · Jul 10, 2024 · 279da70 · 279da70
1 parent 401dd54
commit 279da70
Show file tree

Hide file tree

Showing 14 changed files with 1,444 additions and 1,445 deletions.
diff --git a/Cargo.lock b/Cargo.lock
diff --git a/accounts-db/Cargo.toml b/accounts-db/Cargo.toml
@@ -39,7 +39,7 @@ solana-frozen-abi = { workspace = true, optional = true }
 solana-frozen-abi-macro = { workspace = true, optional = true }
 solana-inline-spl = { workspace = true }
 solana-measure = { workspace = true }
-solana-metrics = { workspace = true }
+# solana-metrics = { workspace = true }
 solana-nohash-hasher = { workspace = true }
 solana-rayon-threadlimit = { workspace = true }
 solana-sdk = { workspace = true }

diff --git a/accounts-db/src/accounts_cache.rs b/accounts-db/src/accounts_cache.rs
@@ -40,25 +40,25 @@ impl Drop for SlotCacheInner {
 
 impl SlotCacheInner {
     pub fn report_slot_store_metrics(&self) {
-        datapoint_info!(
-            "slot_repeated_writes",
-            (
-                "same_account_writes",
-                self.same_account_writes.load(Ordering::Relaxed),
-                i64
-            ),
-            (
-                "same_account_writes_size",
-                self.same_account_writes_size.load(Ordering::Relaxed),
-                i64
-            ),
-            (
-                "unique_account_writes_size",
-                self.unique_account_writes_size.load(Ordering::Relaxed),
-                i64
-            ),
-            ("size", self.size.load(Ordering::Relaxed), i64)
-        );
+        // datapoint_info!(
+        //     "slot_repeated_writes",
+        //     (
+        //         "same_account_writes",
+        //         self.same_account_writes.load(Ordering::Relaxed),
+        //         i64
+        //     ),
+        //     (
+        //         "same_account_writes_size",
+        //         self.same_account_writes_size.load(Ordering::Relaxed),
+        //         i64
+        //     ),
+        //     (
+        //         "unique_account_writes_size",
+        //         self.unique_account_writes_size.load(Ordering::Relaxed),
+        //         i64
+        //     ),
+        //     ("size", self.size.load(Ordering::Relaxed), i64)
+        // );
     }
 
     pub fn get_all_pubkeys(&self) -> Vec<Pubkey> {
@@ -191,21 +191,21 @@ impl AccountsCache {
         self.total_size.load(Ordering::Relaxed)
     }
     pub fn report_size(&self) {
-        datapoint_info!(
-            "accounts_cache_size",
-            (
-                "num_roots",
-                self.maybe_unflushed_roots.read().unwrap().len(),
-                i64
-            ),
-            ("num_slots", self.cache.len(), i64),
-            (
-                "total_unique_writes_size",
-                self.unique_account_writes_size(),
-                i64
-            ),
-            ("total_size", self.size(), i64),
-        );
+        // datapoint_info!(
+        //     "accounts_cache_size",
+        //     (
+        //         "num_roots",
+        //         self.maybe_unflushed_roots.read().unwrap().len(),
+        //         i64
+        //     ),
+        //     ("num_slots", self.cache.len(), i64),
+        //     (
+        //         "total_unique_writes_size",
+        //         self.unique_account_writes_size(),
+        //         i64
+        //     ),
+        //     ("total_size", self.size(), i64),
+        // );
     }
 
     pub fn store(&self, slot: Slot, pubkey: &Pubkey, account: AccountSharedData) -> CachedAccount {