From fd4ea8945410036e551a6905bcc1d7ab24388242 Mon Sep 17 00:00:00 2001 From: Nga Tran Date: Fri, 13 Aug 2021 12:51:37 -0400 Subject: [PATCH] chore: revert panic commit --- data_types/src/partition_metadata.rs | 16 +++++++--------- 1 file changed, 7 insertions(+), 9 deletions(-) diff --git a/data_types/src/partition_metadata.rs b/data_types/src/partition_metadata.rs index 111bb48c00..804db703d1 100644 --- a/data_types/src/partition_metadata.rs +++ b/data_types/src/partition_metadata.rs @@ -1,6 +1,7 @@ //! This module contains structs that describe the metadata for a partition //! including schema, summary statistics, and file locations in storage. +use observability_deps::tracing::warn; use serde::{Deserialize, Serialize}; use std::{ borrow::{Borrow, Cow}, @@ -106,15 +107,12 @@ impl TableSummary { // Validate that the counts are consistent across columns for c in &self.columns { - assert_eq!( - c.total_count(), - count, - "Mismatch counts in table {} column {}, expected {} got {}", - self.name, - c.name, - count, - c.total_count() - ) + // Restore to assert when https://github.com/influxdata/influxdb_iox/issues/2124 is fixed + if c.total_count() != count { + warn!(table_name=%self.name, column_name=%c.name, + column_count=c.total_count(), previous_count=count, + "Mismatch in statistics count, see #2124"); + } } count }