astral-sh / astral-sh/tar-codec

Use rustc-hash?

Open
#9 0 comments 0 reactions 0 assignees View on GitHub
Dominant language
Rust
Stars
9
Forks
3
Avg merge
9m
Merged PRs (30d)
3

Description

Hashing doesn't dominate our performance whatsoever, but might be a small boost to use a non-HashDoS-resistant hashmap. Doing so would be fine w/r/t security, since DoS isn't within our security model.

Contributor guide

Open the contributing guide

Research direction

The issue names no files, tests, or entry points. Start by locating hashmap usage in the Rust encoder and decoder, then compare the current implementation with rustc-hash using relevant performance measurements. Done means establishing whether the change provides a worthwhile boost without exceeding the stated security model.

Written by the indexing model from the issue text.

Assessment

Tech stack
rust
Domain
performance
Issue type
Refactor
Difficulty
4/5
Estimated time
3-5 days
Activity status
Quiet
Clarity
Needs clarification
Newbie friendliness
35/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.