[ENH] Put both token id and token str in the statistics

Author: rescrv

rust/worker/src/execution/functions/statistics.rs

@@ -65,7 +65,7 @@ enum StatisticsValue {
     /// String metadata value associated with a record.
     Str(String),
     /// Sparse vector index observed in metadata.
-    SparseVector(u32),
+    SparseVector(u32, Option<String>),
 }
 
 impl StatisticsValue {
@@ -76,7 +76,7 @@ impl StatisticsValue {
             Self::Int(_) => "int",
             Self::Float(_) => "float",
             Self::Str(_) => "str",
-            Self::SparseVector(_) => "sparse",
+            Self::SparseVector(_, _) => "sparse",
         }
     }
 
@@ -87,12 +87,12 @@ impl StatisticsValue {
             Self::Int(_) => "i",
             Self::Float(_) => "f",
             Self::Str(_) => "s",
-            Self::SparseVector(_) => "sv",
+            Self::SparseVector(_, _) => "sv",
         }
     }
 
     /// A stable representation of the statistics's value.
-    fn stable_value(&self) -> String {
+    fn stable_value1(&self) -> String {
         match self {
             Self::Bool(b) => {
                 format!("{b}")
@@ -102,16 +102,27 @@ impl StatisticsValue {
             }
             Self::Str(s) => s.clone(),
             Self::Float(f) => format!("{f:.16e}"),
-            Self::SparseVector(index) => {
+            Self::SparseVector(index, _) => {
                 format!("{index}")
             }
         }
     }
 
+    /// A stable representation of the statistics's value.
+    fn stable_value2(&self) -> Option<String> {
+        match self {
+            Self::Bool(_) => None,
+            Self::Int(_) => None,
+            Self::Str(_) => None,
+            Self::Float(_) => None,
+            Self::SparseVector(_, token) => token.clone(),
+        }
+    }
+
     /// A stable string representation of a statistics value with type tag.
     /// Separate so display repr can change.
-    fn stable_string(&self) -> String {
-        format!("{}:{}", self.type_prefix(), self.stable_value())
+    fn stable_string1(&self) -> String {
+        format!("{}:{}", self.type_prefix(), self.stable_value1())
     }
 
     /// Convert MetadataValue to a vector of StatisticsValue.
@@ -122,18 +133,31 @@ impl StatisticsValue {
             MetadataValue::Int(i) => vec![StatisticsValue::Int(*i)],
             MetadataValue::Float(f) => vec![StatisticsValue::Float(*f)],
             MetadataValue::Str(s) => vec![StatisticsValue::Str(s.clone())],
-            MetadataValue::SparseVector(sparse) => sparse
-                .indices
-                .iter()
-                .map(|index| StatisticsValue::SparseVector(*index))
-                .collect(),
+            MetadataValue::SparseVector(sparse) => {
+                if let Some(tokens) = sparse.tokens.as_ref() {
+                    sparse
+                        .indices
+                        .iter()
+                        .zip(tokens.iter())
+                        .map(|(index, token)| {
+                            StatisticsValue::SparseVector(*index, Some(token.clone()))
+                        })
+                        .collect()
+                } else {
+                    sparse
+                        .indices
+                        .iter()
+                        .map(|index| StatisticsValue::SparseVector(*index, None))
+                        .collect()
+                }
+            }
         }
     }
 }
 
 impl std::fmt::Display for StatisticsValue {
     fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
-        write!(f, "{}", self.stable_string())
+        write!(f, "{}", self.stable_string1())
     }
 }
 
@@ -144,7 +168,9 @@ impl PartialEq for StatisticsValue {
             (Self::Int(lhs), Self::Int(rhs)) => lhs == rhs,
             (Self::Float(lhs), Self::Float(rhs)) => lhs.to_bits() == rhs.to_bits(),
             (Self::Str(lhs), Self::Str(rhs)) => lhs == rhs,
-            (Self::SparseVector(lhs), Self::SparseVector(rhs)) => lhs == rhs,
+            (Self::SparseVector(lhs1, lhs2), Self::SparseVector(rhs1, rhs2)) => {
+                lhs1 == rhs1 && lhs2 == rhs2
+            }
             _ => false,
         }
     }
@@ -160,7 +186,10 @@ impl Hash for StatisticsValue {
             StatisticsValue::Int(value) => value.hash(state),
             StatisticsValue::Float(value) => value.to_bits().hash(state),
             StatisticsValue::Str(value) => value.hash(state),
-            StatisticsValue::SparseVector(value) => value.hash(state),
+            StatisticsValue::SparseVector(value, token) => {
+                value.hash(state);
+                token.hash(state);
+            }
         }
     }
 }
@@ -202,10 +231,10 @@ impl AttachedFunctionExecutor for StatisticsFunctionExecutor {
         let mut records = Vec::with_capacity(counts.len());
         for (key, inner_map) in counts.into_iter() {
             for (stats_value, count) in inner_map.into_iter() {
-                let stable_value = stats_value.stable_value();
-                let stable_string = stats_value.stable_string();
-                let record_id = format!("{key}::{stable_string}");
-                let document = format!("statistics about {key} for {stable_string}");
+                let stable_value1 = stats_value.stable_value1();
+                let stable_string1 = stats_value.stable_string1();
+                let record_id = format!("{key}::{stable_string1}");
+                let document = format!("statistics about {key} for {stable_string1}");
 
                 let mut metadata = HashMap::with_capacity(4);
                 metadata.insert("count".to_string(), count.output());
@@ -214,7 +243,13 @@ impl AttachedFunctionExecutor for StatisticsFunctionExecutor {
                     "type".to_string(),
                     UpdateMetadataValue::Str(stats_value.stable_type().to_string()),
                 );
-                metadata.insert("value".to_string(), UpdateMetadataValue::Str(stable_value));
+                metadata.insert("value".to_string(), UpdateMetadataValue::Str(stable_value1));
+                if let Some(stable_value2) = stats_value.stable_value2() {
+                    metadata.insert(
+                        "value2".to_string(),
+                        UpdateMetadataValue::Str(stable_value2),
+                    );
+                }
 
                 keys.insert(record_id.clone());
                 records.push(LogRecord {