Merge pull request #2287 from influxdata/ntran/revert

chore: revert panic commit
pull/24376/head
kodiakhq[bot] 2021-08-13 17:48:04 +00:00 committed by GitHub
commit 0aa81bf405
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 7 additions and 9 deletions

View File

@ -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
}