migrate to rand 0.9.0

[coszio]

This migrates the usage of rand to what it expects in the new version, so that it implements the right traits that have been changed. Here is the migration guide.

Before this, it was not possible to use this crate alongside a newer rand version. After this, this crate will probably not be backward compatible with earlier rand versions

[rkuris]

+1 please merge this so we can use the latest rand

[tisonkun]

cc @jonhoo

[rkuris]

+1 please merge this so we can use the latest rand

No longer needed, migrated to rand_distr

[tisonkun]

@rkuris Interesting. Could you share the PR for migration? I may follow your solution :P

[rkuris]

See ava-labs/firewood#775 This other zipf library in rand_distr only works with floats but there is a comment in the docs that you can cast it to integers if needed, so we did that and it seems to work fine.

…

On Mon, Feb 3, 2025, 4:42 PM tison ***@***.***> wrote: @rkuris <https://github.com/rkuris> Interesting. Could you share the PR for migration? I may follow your solution :P — Reply to this email directly, view it on GitHub <#12 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAYLR3BTJ5GWMAOL5B3P5TL2OAEH7AVCNFSM6AAAAABWBJ4XCSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMZSGUYDEMBUGY> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[tisonkun]

Thanks. I tried it at foyer-rs/foyer#854.

[jonhoo]

Sorry for the delay! Now that there is Zipf support directly in rand_distr (as of rust-random/rand#1136), that's probably the one to use! I had generally hoped that rand would move away from "all the distributions in one crate", but given that they're moving the other direction, I think it probably doesn't make sense to maintain this separately any more, so I'll go ahead and deprecate it. Especially since it seems rand uses effectively the same method.