Fix handling of components that only vary by descriptors

Cargo.lock

@@ -8549,7 +8549,7 @@ dependencies = [
  "re_chunk_store",
  "re_format",
  "re_log_types",
- "re_types",
+ "re_types_core",
  "re_ui",
  "re_viewer_context",
 ]
@@ -9676,7 +9676,6 @@ dependencies = [
  "re_test_context",
  "re_tracing",
  "re_types",
- "re_types_core",
  "re_ui",
  "re_viewer_context",
  "re_viewport_blueprint",

crates/store/re_chunk/examples/chunk_latest_at.rs

@@ -11,13 +11,13 @@ fn main() -> anyhow::Result<()> {
     let query = LatestAtQuery::new(TimelineName::new("frame"), 4);
 
     // Find all relevant data for a query:
-    let chunk = chunk.latest_at(&query, &MyPoints::descriptor_points());
+    let chunk = chunk.latest_at(&query, MyPoints::descriptor_points().component);
     eprintln!("{:?} @ {query:?}:\n{chunk}", MyPoints::descriptor_points());
 
     // And then slice it as appropriate:
     let chunk = chunk
         .timeline_sliced(TimelineName::log_time())
-        .component_sliced(&MyPoints::descriptor_points());
+        .component_sliced(MyPoints::descriptor_points().component);
     eprintln!("Sliced down to specific timeline and component:\n{chunk}");
 
     Ok(())

crates/store/re_chunk/examples/chunk_range.rs

@@ -14,13 +14,13 @@ fn main() -> anyhow::Result<()> {
     let query = RangeQuery::new(TimelineName::new("frame"), AbsoluteTimeRange::EVERYTHING);
 
     // Find all relevant data for a query:
-    let chunk = chunk.range(&query, &MyPoints::descriptor_points());
+    let chunk = chunk.range(&query, MyPoints::descriptor_points().component);
     eprintln!("{:?} @ {query:?}:\n{chunk}", MyPoints::descriptor_points());
 
     // And then slice it as appropriate:
     let chunk = chunk
         .timeline_sliced(TimelineName::log_time())
-        .component_sliced(&MyPoints::descriptor_points());
+        .component_sliced(MyPoints::descriptor_points().component);
     eprintln!("Sliced down to specific timeline and component:\n{chunk}");
 
     Ok(())

crates/store/re_chunk/src/builder.rs

@@ -5,7 +5,7 @@ use nohash_hasher::IntMap;
 use re_log_types::{EntityPath, NonMinI64, TimePoint, Timeline, TimelineName};
 use re_types_core::{AsComponents, ComponentBatch, ComponentDescriptor, SerializedComponentBatch};
 
-use crate::{Chunk, ChunkId, ChunkResult, RowId, TimeColumn, chunk::ChunkComponents};
+use crate::{Chunk, ChunkId, ChunkResult, RowId, TimeColumn};
 
 // ---
 
@@ -297,16 +297,14 @@ impl ChunkBuilder {
 
         let components = {
             re_tracing::profile_scope!("components");
-            ChunkComponents(
-                components
-                    .into_iter()
-                    .filter_map(|(component_desc, arrays)| {
-                        let arrays = arrays.iter().map(|array| array.as_deref()).collect_vec();
-                        re_arrow_util::arrays_to_list_array_opt(&arrays)
-                            .map(|list_array| (component_desc, list_array))
-                    })
-                    .collect(),
-            )
+            components
+                .into_iter()
+                .filter_map(|(component_desc, arrays)| {
+                    let arrays = arrays.iter().map(|array| array.as_deref()).collect_vec();
+                    re_arrow_util::arrays_to_list_array_opt(&arrays)
+                        .map(|list_array| (component_desc, list_array))
+                })
+                .collect()
         };
 
         Chunk::from_native_row_ids(id, entity_path, None, &row_ids, timelines, components)
@@ -350,23 +348,21 @@ impl ChunkBuilder {
                 .map(|(timeline, time_column)| (timeline, time_column.build()))
                 .collect(),
             {
-                ChunkComponents(
-                    components
-                        .into_iter()
-                        .filter_map(|(component_desc, arrays)| {
-                            let arrays = arrays.iter().map(|array| array.as_deref()).collect_vec();
-                            // If we know the datatype in advance, we're able to keep even fully sparse
-                            // columns around.
-                            if let Some(datatype) = datatypes.get(&component_desc) {
-                                re_arrow_util::arrays_to_list_array(datatype.clone(), &arrays)
-                                    .map(|list_array| (component_desc, list_array))
-                            } else {
-                                re_arrow_util::arrays_to_list_array_opt(&arrays)
-                                    .map(|list_array| (component_desc, list_array))
-                            }
-                        })
-                        .collect(),
-                )
+                components
+                    .into_iter()
+                    .filter_map(|(component_desc, arrays)| {
+                        let arrays = arrays.iter().map(|array| array.as_deref()).collect_vec();
+                        // If we know the datatype in advance, we're able to keep even fully sparse
+                        // columns around.
+                        if let Some(datatype) = datatypes.get(&component_desc) {
+                            re_arrow_util::arrays_to_list_array(datatype.clone(), &arrays)
+                                .map(|list_array| (component_desc, list_array))
+                        } else {
+                            re_arrow_util::arrays_to_list_array_opt(&arrays)
+                                .map(|list_array| (component_desc, list_array))
+                        }
+                    })
+                    .collect()
             },
         )
     }