HdrHistogram / HdrHistogram/HdrHistogram_rust
Make public deserialize methods `deser_v2` & `deser_v2_compressed`
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 372
- Forks
- 50
- PR merge metrics
- No merged PRs in 30d
Description
Specifically I'm interested in calling the deser_v2 method directly. My motivation is I'm using hdrhistogram in a WASM module where I only need to deserialize uncompressed histograms. By using the deserialize method the compiler can not statically determine that I will not be deserializing compressed histograms and flate2 gets included in my WASM.
I only included deser_v2_compressed in the title because I figured if you make public one of them then perhaps it makes sense to do both(??).
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the public deserialize API and the deser_v2 and deser_v2_compressed methods in the Rust crate. Check how deserialize selects compressed versus uncompressed decoding, then make the requested methods directly callable and verify that uncompressed WASM use does not require compressed deserialization.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust, wasm
- Domain
- api
- Issue type
- Feature
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100