fix(bloom): align MAX_NUM_BITS with java implementation

Author: notfilippo

datasketches/src/bloom/builder.rs

@@ -16,10 +16,11 @@
 // under the License.
 
 use super::BloomFilter;
+use crate::bloom::sketch::PREAMBLE_LONGS_STANDARD;
 use crate::hash::DEFAULT_UPDATE_SEED;
 
 const MIN_NUM_BITS: u64 = 64;
-const MAX_NUM_BITS: u64 = (1u64 << 35) - 64; // ~32 GB - reasonable limit
+const MAX_NUM_BITS: u64 = (i32::MAX as u64 - PREAMBLE_LONGS_STANDARD as u64) * 64;
 
 /// Builder for creating [`BloomFilter`] instances.
 ///

datasketches/src/bloom/sketch.rs

@@ -24,8 +24,8 @@ use crate::error::Error;
 use crate::hash::XxHash64;
 
 // Serialization constants
+pub const PREAMBLE_LONGS_STANDARD: u8 = 4;
 const PREAMBLE_LONGS_EMPTY: u8 = 3;
-const PREAMBLE_LONGS_STANDARD: u8 = 4;
 const BLOOM_FAMILY_ID: u8 = 21; // Bloom filter family ID
 const SERIAL_VERSION: u8 = 1;
 const EMPTY_FLAG_MASK: u8 = 1 << 2;