Remove dead code: timepoint_max (#10008)

Unused code.

Author: emilk

crates/store/re_chunk/src/chunk.rs

@@ -15,7 +15,7 @@ use nohash_hasher::IntMap;
 use re_arrow_util::ArrowArrayDowncastRef as _;
 use re_byte_size::SizeBytes as _;
 use re_log_types::{
-    EntityPath, NonMinI64, ResolvedTimeRange, TimeInt, TimePoint, TimeType, Timeline, TimelineName,
+    EntityPath, NonMinI64, ResolvedTimeRange, TimeInt, TimeType, Timeline, TimelineName,
 };
 use re_types_core::{
     ArchetypeName, ComponentDescriptor, ComponentName, DeserializationError, Loggable as _,
@@ -1240,16 +1240,6 @@ impl Chunk {
     pub fn components(&self) -> &ChunkComponents {
         &self.components
     }
-
-    /// Computes the maximum value for each and every timeline present across this entire chunk,
-    /// and returns the corresponding [`TimePoint`].
-    #[inline]
-    pub fn timepoint_max(&self) -> TimePoint {
-        self.timelines
-            .values()
-            .map(|info| (info.timeline, info.time_range.max()))
-            .collect()
-    }
 }
 
 impl std::fmt::Display for Chunk {

crates/store/re_chunk/src/transport.rs

@@ -242,7 +242,6 @@ impl Chunk {
     pub fn from_arrow_msg(msg: &re_log_types::ArrowMsg) -> ChunkResult<Self> {
         let re_log_types::ArrowMsg {
             chunk_id: _,
-            timepoint_max: _,
             batch,
             on_release: _,
         } = msg;
@@ -256,8 +255,7 @@ impl Chunk {
         self.sanity_check()?;
 
         Ok(re_log_types::ArrowMsg {
-            chunk_id: re_tuid::Tuid::from_u128(self.id().as_u128()),
-            timepoint_max: self.timepoint_max(),
+            chunk_id: self.id().as_tuid(),
             batch: self.to_record_batch()?,
             on_release: None,
         })

crates/store/re_log_types/src/arrow_msg.rs

@@ -7,8 +7,6 @@ use std::sync::Arc;
 
 use arrow::array::RecordBatch as ArrowRecordBatch;
 
-use crate::TimePoint;
-
 /// An arbitrary callback to be run when an [`ArrowMsg`], and more specifically the
 /// [`ArrowRecordBatch`] within it, goes out of scope.
 ///
@@ -72,12 +70,6 @@ pub struct ArrowMsg {
     /// Unique identifier for the chunk in this message.
     pub chunk_id: re_tuid::Tuid,
 
-    /// The maximum values for all timelines across the entire batch of data.
-    ///
-    /// Used to timestamp the batch as a whole for e.g. latency measurements without having to
-    /// deserialize the arrow payload.
-    pub timepoint_max: TimePoint,
-
     /// Schema and data for all control & data columns.
     pub batch: ArrowRecordBatch,
 

crates/store/re_log_types/src/lib.rs

@@ -784,12 +784,11 @@ impl SizeBytes for ArrowMsg {
     fn heap_size_bytes(&self) -> u64 {
         let Self {
             chunk_id,
-            timepoint_max,
             batch,
             on_release: _,
         } = self;
 
-        chunk_id.heap_size_bytes() + timepoint_max.heap_size_bytes() + batch.heap_size_bytes()
+        chunk_id.heap_size_bytes() + batch.heap_size_bytes()
     }
 }
 

crates/top/rerun/src/commands/rrd/migrate.rs

@@ -120,7 +120,6 @@ fn migrate_from_to(from_path: &Utf8PathBuf, to_path: &Utf8PathBuf) -> anyhow::Re
                             store_id,
                             re_log_types::ArrowMsg {
                                 chunk_id: arrow_msg.chunk_id,
-                                timepoint_max: arrow_msg.timepoint_max.clone(),
                                 batch,
                                 on_release: None,
                             },