[GH-ISSUE #365] Evaluate all Vec usage and replace with SmallVec where apropriate #463

New issue

Open

opened 2026-03-15 22:39:51 +03:00 by kerem · 3 comments

kerem commented

2026-03-15 22:39:51 +03:00

Owner

Originally created by @bluejekyll on GitHub (Mar 27, 2018).
Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/365

Make sure to run benchmarks before and after to see if there is a difference ;)

This should reduce heap allocations throughout the various libraries.

Originally created by @bluejekyll on GitHub (Mar 27, 2018). Original GitHub issue: https://github.com/hickory-dns/hickory-dns/issues/365 Make sure to run benchmarks before and after to see if there is a difference ;) This should reduce heap allocations throughout the various libraries.

kerem added the

crate:all

cleanup

easy

labels

2026-03-15 22:39:51 +03:00

kerem commented

2026-03-15 22:40:01 +03:00

Author

Owner

@link2xt commented on GitHub (Nov 23, 2021):

For some reason there is also a tinyvec dependency already. tinyvec is a safer alternative to smallvec:
https://www.reddit.com/r/rust/comments/n2429h/arrayvec_vs_smallvec_vs_tinyvec/

Which one to use depends on actual performance of course, but would be nice to choose one and have a single dependency.

@link2xt commented on GitHub (Nov 23, 2021): For some reason there is also a `tinyvec` dependency already. `tinyvec` is a safer alternative to `smallvec`: https://www.reddit.com/r/rust/comments/n2429h/arrayvec_vs_smallvec_vs_tinyvec/ Which one to use depends on actual performance of course, but would be nice to choose one and have a single dependency.

kerem commented

2026-03-15 22:40:07 +03:00

Author

Owner

@djc commented on GitHub (Nov 24, 2021):

See some discussion in https://github.com/bluejekyll/trust-dns/pull/1388.

If you want to rip out uses of smallvec in favor of tinyvec, that'd be great!

@djc commented on GitHub (Nov 24, 2021): See some discussion in https://github.com/bluejekyll/trust-dns/pull/1388. If you want to rip out uses of smallvec in favor of tinyvec, that'd be great!

kerem commented

2026-03-15 22:40:12 +03:00

Author

Owner

@darnuria commented on GitHub (Jun 20, 2023):

Hi @bluejekyll /@djc, have some time this end of the month if still pertinent i can try to claim the task and do something to only use tinyvec ! :)

(maybe a renaming of the ticket or opening a new one if neccessary)

@darnuria commented on GitHub (Jun 20, 2023): Hi @bluejekyll /@djc, have some time this end of the month if still pertinent i can try to claim the task and do something to only use `tinyvec` ! :) (maybe a renaming of the ticket or opening a new one if neccessary)

kerem referenced this issue

2026-03-15 22:47:07 +03:00

[GH-ISSUE #463] Cleanup DnsHandle to allow associated return type for send() #494