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

Open
opened 2026-03-07 22:36:54 +03:00 by kerem · 3 comments
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.
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.

<!-- gh-comment-id:976431772 --> @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.
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!

<!-- gh-comment-id:978081487 --> @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!
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)

<!-- gh-comment-id:1599604397 --> @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)
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/hickory-dns#167
No description provided.