search

tikv / jemallocator

Rust allocator using jemalloc as a backend
Other
364 stars 62 forks source link

Replace all `tikv-jemallocator` to `jemallocator` #63

Open Xuanwo opened 1 year ago

Xuanwo commented 1 year ago

Since tikv/jemallocator has become the upstream of jemallocator, I believe it's time to replace all tikv-jemallocator to jemallocator to avoid confusion.

And we should push a new version to tikv-jemallocator with notes that migrate to jemallocator instead.

BusyJay commented 1 year ago

See #56. I'm afraid we have to keep using tikv-jemallocator.

Xuanwo commented 1 year ago

Hi @gnzlbg, would you like to add either @BusyJay or @tikv/maintainers as a co-owner of the jemallocator-global crate? Thanks in advance.

Xuanwo commented 1 year ago

There is an another option that merge jemallocator-global features into jemallocator (maybe under a feature flag). jemallocator-global is simple and no extra deps. I'm willing to take this issue if it looks good to you.

BusyJay commented 1 year ago

That might work too. However, we have made the claim in the last release that tikv-jemallocator is preferred than jemallocator for new projects. Changing it back and forth may cause more confusions.

On the other hand, even jemallocator is chosen, we will still need to publish two versions till next minor bump, which is unlikely happen until next jemalloc breaking change is released.