ZENOTME commented on code in PR #77: URL: https://github.com/apache/datasketches-rust/pull/77#discussion_r2808598552
########## datasketches/src/hash/mod.rs: ########## @@ -35,7 +35,8 @@ pub(crate) use self::xxhash::XxHash64; /// and seed are identical for both sketches, otherwise the assumed 1:1 relationship between the /// original source key value and the hashed bit string would be violated. Once you have developed /// a history of stored sketches you are stuck with it. -pub(crate) const DEFAULT_UPDATE_SEED: u64 = 9001; +/// The default seed used for hashing. +pub const DEFAULT_UPDATE_SEED: u64 = 9001; Review Comment: It was used in `theta_serialization_test` because the current interface is the user reponsiblity to provide the hash seed, so we need to expose it `fn deserialize(bytes: &[u8], expected_seed: u64)`. I realized that fn deserialize(bytes: &[u8], expected_seed: Option<u64>) may be a better choice, if expected_seed is None, then we will use the default seed and don't need to expose it directly. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
