From 03c1a07752b3ab073df183664b870c396c2c0ed3 Mon Sep 17 00:00:00 2001
From: allada <thegreatall@gmail.com>
Date: Fri, 8 Sep 2023 19:22:39 -0500
Subject: [PATCH] Make stats in EvictingMap more useful

Changes some of the stats published in prometheus in EvictingMap
to be more useful.
---
 util/evicting_map.rs | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/util/evicting_map.rs b/util/evicting_map.rs
index 3c582729a..05235420f 100644
--- a/util/evicting_map.rs
+++ b/util/evicting_map.rs
@@ -385,18 +385,19 @@ impl<T: LenEntry + Debug, I: InstantWrapper> MetricsComponent for EvictingMap<T,
                 &state
                     .lru
                     .peek_lru()
-                    .map(|(_, v)| v.seconds_since_anchor as i64 + self.anchor_time.unix_timestamp() as i64)
+                    .map(|(_, v)| self.anchor_time.unix_timestamp() as i64 - v.seconds_since_anchor as i64)
                     .unwrap_or(-1),
                 "Timestamp of the oldest item in the store",
             );
             c.publish(
-                "oldest_item_age_nanos",
+                "newest_item_timestamp",
                 &state
                     .lru
-                    .peek_lru()
-                    .map(|(_, v)| v.seconds_since_anchor as i64 + self.anchor_time.elapsed().as_secs() as i64)
+                    .iter()
+                    .next()
+                    .map(|(_, v)| self.anchor_time.unix_timestamp() as i64 - v.seconds_since_anchor as i64)
                     .unwrap_or(-1),
-                "Age of the oldest item in the store in seconds",
+                "Timestamp of the newest item in the store",
             );
             c.publish(
                 "evicted_items_total",
@@ -435,8 +436,8 @@ impl<T: LenEntry + Debug, I: InstantWrapper> MetricsComponent for EvictingMap<T,
             );
             c.publish_stats(
                 "item_size_bytes",
-                state.lru.iter().take(10_000).map(|(_, v)| v.data.len()),
-                "Stats about the first 10_000 items in the store (these are oldest items in the store)",
+                state.lru.iter().take(1_000_000).map(|(_, v)| v.data.len()),
+                "Stats about the first 1_000_000 items in the store (these are newest items in the store)",
             );
         });
     }