search

domodwyer / bloom2

A fast 2-level, sparse bloom filter implementation
BSD 3-Clause "New" or "Revised" License
12 stars 1 forks source link

Non-serde serialization? #16

Open siennathesane opened 1 month ago

siennathesane commented 1 month ago

I'm currently using bloom2 in a performance sensitive area and serde is not a good fit. Would you mind if I expose the fields via getters, swap out Vec for Bytes, and add some functions to the builders to accept some pre-defined fields? I'll leave the existing serde functionality in-place and make sure it still works as expected

domodwyer commented 1 month ago

Hey @siennathesane - I'd be super interested in seeing how this would look!

I think we'd want to maintain the performance of the current implementation, but I'd expect that's doable with what you propose.