Skip to content

Commit 9392d19

Browse files
Enable union-merge test for precisions 16-18 now that correction tables cover them
1 parent 6eb36dd commit 9392d19

1 file changed

Lines changed: 0 additions & 8 deletions

File tree

tests/test_hll.rs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -133,14 +133,6 @@ pub fn test_union_merge_at_precision_and_bits<
133133
R: Registers<P, B>,
134134
H: HasherType,
135135
>() {
136-
// The hash-list cardinality correction tables only cover precisions 4..=15, so
137-
// `estimate_cardinality` in hash-list mode panics at precisions 16..=18 (a pre-existing
138-
// gap in the generated correction coefficients, unrelated to the merger). We restrict
139-
// this test to the supported range.
140-
if P::EXPONENT > 15 {
141-
return;
142-
}
143-
144136
let mut left: HyperLogLog<P, B, R, H> = Default::default();
145137
let mut right: HyperLogLog<P, B, R, H> = Default::default();
146138
let mut exact_left = std::collections::HashSet::new();

0 commit comments

Comments
 (0)