Skip to content

[EXPERIMENT] [DONT MERGE] updates and fastmurmur3 as hashing crate#29

Open
sassman wants to merge 13 commits intomasterfrom
feat/fastmurmur3-and-dependency-update
Open

[EXPERIMENT] [DONT MERGE] updates and fastmurmur3 as hashing crate#29
sassman wants to merge 13 commits intomasterfrom
feat/fastmurmur3-and-dependency-update

Conversation

@sassman
Copy link

@sassman sassman commented Feb 2, 2026

Performance & Dependencies

  • Replaced hashing implementation with fastmurmur3 for better performance (that it's better is not yet verified!)
  • Refactored hash-related structs into dedicated hash.rs module
  • Updated parking_lot dependency to 0.12
  • Added CSV benchmark suite to verify performance improvements

Build & Development

  • Pinned nightly Rust toolchain via rust-toolchain.toml
  • Improved build.rs with better error handling and documentation
  • Fixed hard-coded paths, added ANTLR download instructions
  • Added tools/ directory for external jars, updated .gitignore

Code Quality

  • Multiple clippy lint fixes across codebase
  • Ran cargo fmt on entire project
  • Cleaned up unnecessary trait bounds and implementations
  • Removed unused imports and dead code paths

Generated Code

  • Regenerated test parsers (CSV, Labels, XML, etc.) with updated ANTLR snapshot version
  • Updated generated code formatting to match current standards

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant