Deprecate legacy Points{2|3}D and all their components

crates/re_arrow_store/benches/arrow2.rs

@@ -10,10 +10,13 @@ use criterion::{criterion_group, Criterion};
 use itertools::Itertools;
 use re_components::{
     datagen::{build_some_instances, build_some_point2d, build_some_rects},
-    Point2D, Rect2D,
+    Rect2D,
 };
 use re_log_types::{DataCell, SizeBytes as _};
-use re_types::{components::InstanceKey, Component};
+use re_types::{
+    components::{InstanceKey, Point2D},
+    Component,
+};
 
 // ---
 

crates/re_arrow_store/examples/latest_component.rs

@@ -8,10 +8,13 @@ use re_arrow_store::polars_util::latest_component;
 use re_arrow_store::{test_row, DataStore, LatestAtQuery, TimeType, Timeline};
 use re_components::{
     datagen::{build_frame_nr, build_some_point2d, build_some_rects},
-    Point2D, Rect2D,
+    Rect2D,
 };
 use re_log_types::EntityPath;
-use re_types::{components::InstanceKey, Loggable};
+use re_types::{
+    components::{InstanceKey, Point2D},
+    Loggable,
+};
 
 fn main() {
     let mut store = DataStore::new(InstanceKey::name(), Default::default());

crates/re_arrow_store/examples/latest_components.rs

@@ -10,10 +10,13 @@ use re_arrow_store::polars_util::latest_components;
 use re_arrow_store::{test_row, DataStore, LatestAtQuery, TimeType, Timeline};
 use re_components::{
     datagen::{build_frame_nr, build_some_point2d, build_some_rects},
-    Point2D, Rect2D,
+    Rect2D,
 };
 use re_log_types::EntityPath;
-use re_types::{components::InstanceKey, Loggable};
+use re_types::{
+    components::{InstanceKey, Point2D},
+    Loggable,
+};
 
 fn main() {
     let mut store = DataStore::new(InstanceKey::name(), Default::default());

crates/re_arrow_store/examples/range_components.rs

@@ -8,10 +8,13 @@ use polars_core::prelude::JoinType;
 use re_arrow_store::{polars_util, test_row, DataStore, RangeQuery, TimeRange};
 use re_components::{
     datagen::{build_frame_nr, build_some_point2d, build_some_rects},
-    Point2D, Rect2D,
+    Rect2D,
 };
 use re_log_types::{EntityPath, TimeType, Timeline};
-use re_types::{components::InstanceKey, Loggable};
+use re_types::{
+    components::{InstanceKey, Point2D},
+    Loggable as _,
+};
 
 fn main() {
     let mut store = DataStore::new(InstanceKey::name(), Default::default());

crates/re_arrow_store/src/store_write.rs

@@ -85,11 +85,15 @@ impl DataStore {
                 use std::collections::hash_map::Entry;
                 match self.type_registry.entry(cell.component_name()) {
                     Entry::Occupied(entry) => {
-                        if entry.get() != cell.datatype() {
+                        // NOTE: Don't care about extensions until the migration is over (arrow2-convert
+                        // issues).
+                        let expected = entry.get().to_logical_type().clone();
+                        let got = cell.datatype().to_logical_type().clone();
+                        if expected != got {
                             return Err(WriteError::TypeCheck {
                                 component: cell.component_name(),
-                                expected: entry.get().clone(),
-                                got: cell.datatype().clone(),
+                                expected,
+                                got,
                             });
                         }
                     }

crates/re_arrow_store/tests/correctness.rs

@@ -217,7 +217,7 @@ fn range_join_across_single_row_impl(store: &mut DataStore) {
         prelude::{DataFrame, JoinType},
         series::Series,
     };
-    use re_components::{ColorRGBA, Point2D};
+    use re_types::components::{Color, Point2D};
 
     let ent_path = EntityPath::from("this/that");
 
@@ -232,7 +232,7 @@ fn range_join_across_single_row_impl(store: &mut DataStore) {
         timeline_frame_nr,
         re_arrow_store::TimeRange::new(i64::MIN.into(), i64::MAX.into()),
     );
-    let components = [InstanceKey::name(), Point2D::name(), ColorRGBA::name()];
+    let components = [InstanceKey::name(), Point2D::name(), Color::name()];
     let dfs = re_arrow_store::polars_util::range_components(
         store,
         &query,
@@ -247,12 +247,12 @@ fn range_join_across_single_row_impl(store: &mut DataStore) {
         let instances =
             InstanceKey::to_arrow(vec![InstanceKey(0), InstanceKey(1), InstanceKey(2)], None);
         let points = Point2D::to_arrow(points, None);
-        let colors = ColorRGBA::to_arrow(colors, None);
+        let colors = Color::to_arrow(colors, None);
 
         DataFrame::new(vec![
             Series::try_from((InstanceKey::name().as_ref(), instances)).unwrap(),
             Series::try_from((Point2D::name().as_ref(), points)).unwrap(),
-            Series::try_from((ColorRGBA::name().as_ref(), colors)).unwrap(),
+            Series::try_from((Color::name().as_ref(), colors)).unwrap(),
         ])
         .unwrap()
     };