When not going through pre-binned the read bytes are exact

disk/src/binnedstream.rs

@@ -65,6 +65,7 @@ impl BinnedStream {
                         }
                         Ok(PreBinnedItem::RangeComplete) => Some(Ok(MinMaxAvgScalarBinBatchStreamItem::RangeComplete)),
                         Ok(PreBinnedItem::EventDataReadStats(stats)) => {
+                            info!("BinnedStream  observes stats {:?}", stats);
                             Some(Ok(MinMaxAvgScalarBinBatchStreamItem::EventDataReadStats(stats)))
                         }
                         Ok(PreBinnedItem::Log(item)) => Some(Ok(MinMaxAvgScalarBinBatchStreamItem::Log(item))),

disk/src/cache/pbv.rs

@@ -112,7 +112,10 @@ impl PreBinnedValueStream {
             match k {
                 Ok(Values(k)) => Ok(PreBinnedItem::Batch(k)),
                 Ok(RangeComplete) => Ok(PreBinnedItem::RangeComplete),
-                Ok(EventDataReadStats(stats)) => Ok(PreBinnedItem::EventDataReadStats(stats)),
+                Ok(EventDataReadStats(stats)) => {
+                    info!("PreBinnedValueStream  ✙ ✙ ✙ ✙ ✙ ✙ ✙ ✙ ✙ ✙ ✙ ✙ ✙    stats {:?}", stats);
+                    Ok(PreBinnedItem::EventDataReadStats(stats))
+                }
                 Ok(Log(item)) => Ok(PreBinnedItem::Log(item)),
                 Err(e) => Err(e),
             }

disk/src/merge.rs

@@ -198,10 +198,6 @@ where
         }
     }
 
-    // This can:
-    // Do nothing if all have Val or Finished.
-    // But if some is None:
-    // We might get some Pending from upstream. In that case, caller also wants to abort here.
     fn replenish(self: &mut Pin<&mut Self>, cx: &mut Context) -> Poll<Result<(), Error>> {
         use Poll::*;
         let mut pending = 0;