Phaneendra293 opened a new pull request, #19872: URL: https://github.com/apache/datafusion/pull/19872
This PR updates the Clippy configuration to disallow usage of std::collections::HashMap and std::collections::HashSet across the DataFusion codebase. The change helps enforce consistent usage of project-preferred HashMap/set implementations, avoiding accidental use of the standard library types. What changed Updated clippy. toml to add std::collections::HashMap and std::collections::HashSet to the list of disallowed types. Ensures new code follows the project’s hashing and performance conventions. Why The default Rust HashMap / HashSet use randomized hashing, which may be: non-deterministic, sub-optimal for performance-critical paths, inconsistent with DataFusion’s existing design choices. Enforcing this at the lint level prevents regressions and improves code consistency for contributors. Testing No functional code changes. CI / Clippy will fail if std::HashMap or std::HashSet are newly introduced. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
