Don't assert if inserting a rowid with a matching timepoint does not …

…create a conflict (rerun-io#1832)
luxonis · Apr 12, 2023 · c181adf · c181adf
1 parent d2d83d7
commit c181adf
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/crates/re_arrow_store/src/store_write.rs b/crates/re_arrow_store/src/store_write.rs
@@ -245,8 +245,10 @@ impl MetadataRegistry<TimePoint> {
                 let entry = entry.get_mut();
                 for (timeline, time) in timepoint {
                     if let Some(old_time) = entry.insert(timeline, time) {
-                        re_log::error!(%row_id, ?timeline, old_time = ?old_time, new_time = ?time, "detected re-used `RowId/Timeline` pair, this is illegal and will lead to undefined behavior in the datastore");
-                        debug_assert!(false, "detected re-used `RowId/Timeline`");
+                        if old_time != time {
+                            re_log::error!(%row_id, ?timeline, old_time = ?old_time, new_time = ?time, "detected re-used `RowId/Timeline` pair, this is illegal and will lead to undefined behavior in the datastore");
+                            debug_assert!(false, "detected re-used `RowId/Timeline`");
+                        }
                     }
                 }
             }