From 7628e60532d8fe01e85fb34fa98867b7a61c6e7c Mon Sep 17 00:00:00 2001 From: Jeremy Leibs Date: Wed, 12 Apr 2023 15:05:29 -0400 Subject: [PATCH] Don't assert if inserting a rowid with a matching timepoint does not create a conflict (#1832) --- crates/re_arrow_store/src/store_write.rs | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/crates/re_arrow_store/src/store_write.rs b/crates/re_arrow_store/src/store_write.rs index 853bdf4b222c0..d1a49528964c9 100644 --- a/crates/re_arrow_store/src/store_write.rs +++ b/crates/re_arrow_store/src/store_write.rs @@ -245,8 +245,10 @@ impl MetadataRegistry { 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`"); + } } } }