tisonkun commented on PR #84: URL: https://github.com/apache/datasketches-rust/pull/84#issuecomment-3851097618
I'm going to do the following tasks after this patch is merged: 1. `CpcWrapper` to read fields without fully deserializing the sketch. This is implemented in the Java impl as well. 2. Investigate whether we need `introspective_insertion_sort`. Rust's slice sort should properly leverage existing ordered items already. For this patch, one open question is whether to include the decoding table as static values, or build it at the first access (using `OnceLock` or so). I tend to keep the static decoding tables. They should not increase the binary size too much. -- 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]
