Address some of Pawan's review comments

Author: michaelsproul

beacon_node/beacon_chain/src/metrics.rs

@@ -1619,13 +1619,6 @@ pub static BLOBS_SIDECAR_PROCESSING_REQUESTS: LazyLock<Result<IntCounter>> = Laz
         "Count of all blob sidecars submitted for processing",
     )
 });
-pub static BLOBS_COLUMN_SIDECAR_PROCESSING_REQUESTS: LazyLock<Result<IntCounter>> =
-    LazyLock::new(|| {
-        try_create_int_counter(
-            "beacon_blobs_column_sidecar_processing_requests_total",
-            "Count of all data column sidecars submitted for processing",
-        )
-    });
 pub static BLOBS_SIDECAR_PROCESSING_SUCCESSES: LazyLock<Result<IntCounter>> = LazyLock::new(|| {
     try_create_int_counter(
         "beacon_blobs_sidecar_processing_successes_total",

consensus/types/src/runtime_var_list.rs

@@ -46,8 +46,8 @@ pub struct RuntimeVariableList<T> {
 }
 
 impl<T> RuntimeVariableList<T> {
-    /// Returns `Some` if the given `vec` equals the fixed length of `Self`. Otherwise returns
-    /// `None`.
+    /// Returns `Ok` if the given `vec` equals the fixed length of `Self`. Otherwise returns
+    /// `Err(OutOfBounds { .. })`.
     pub fn new(vec: Vec<T>, max_len: usize) -> Result<Self, Error> {
         if vec.len() <= max_len {
             Ok(Self { vec, max_len })