Hi there, good night,
Wanted to know what you guys think about the following PR on hdrhistogram_c
It adds the ability to change the HdrHistogram allocator at compile time. This can be important to integrate into other projects that use by default different allocators like jemalloc ( example of redis ).
I think this is a great idea! Only comment is I would have done it a bit differently, instead of using a compile flag, I'd prefer to make this allocator selection dynamic and have an extra API to customize the malloc/calloc functions and use function pointers that default to the standard malloc/calloc
A question & idea down the path of @filipecosta90 recent efforts towards more efficient percentile computation: where do things stand with regards to packed histogram support in the innards of our various implementations (either in already-there or planning-to-add-soon form)?
@giltene could you summarize what this packed histogram feature is?