Skip to content

perf: parallelize package parsing#109

Open
saurabhhhcodes wants to merge 1 commit into
pie-314:devfrom
saurabhhhcodes:perf/rayon-package-parsing-72
Open

perf: parallelize package parsing#109
saurabhhhcodes wants to merge 1 commit into
pie-314:devfrom
saurabhhhcodes:perf/rayon-package-parsing-72

Conversation

@saurabhhhcodes

Copy link
Copy Markdown

Summary

  • parallelizes stateless APT search/detail/update parsing with Rayon
  • parallelizes pacman installed/update parsing loops
  • switches shared alternating two-line package output parsing to par_chunks(2) while preserving score filtering and sorted results
  • adds focused parser tests for APT line handling and alternating package output chunking

Notes

rayon is already present in the current dev branch's Cargo.toml, so this PR keeps the dependency set unchanged and applies it to the parsing paths requested in #72.

Validation

  • cargo fmt --all -- --check
  • cargo test managers -- --nocapture
  • cargo test
  • cargo clippy --workspace --all-targets --all-features -- -D warnings -A dead_code -A clippy::type_complexity
  • git diff --check

Closes #72

@vercel

vercel Bot commented Jun 1, 2026

Copy link
Copy Markdown

Someone is attempting to deploy a commit to the pie's projects Team on Vercel.

A member of the Team first needs to authorize it.

@saurabhhhcodes

Copy link
Copy Markdown
Author

Validation update for #72:

  • cargo fmt --all -- --check
  • cargo test managers -- --nocapture
  • cargo test
  • cargo clippy --workspace --all-targets --all-features -- -D warnings -A dead_code -A clippy::type_complexity
  • git diff --check

GitHub repo checks are also green: Verify and Build passed. The remaining Vercel status is external team authorization noise, not a Rust/build failure.

If accepted, could you please add the scoring labels from the issue: gssoc:approved, level:intermediate, and type:performance?

@pie-314 pie-314 added the ai-slop label Jun 5, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants