[GitHub] [arrow-datafusion] Dandandan edited a comment on issue #677: Use a single HashMap implementation consistently across the code base

[GitBox <gi...@apache.org>]

Dandandan edited a comment on issue #677:
URL: https://github.com/apache/arrow-datafusion/issues/677#issuecomment-873671789


   The hash brown crate is used  for those reasons:
   
   * It exposes some more APIs like `RawEntry`, which can avoid hashing / looking up elements twice.
   * It is slightly faster than the version included with Rust std, not only because of aHash (vs SipHasher used in std), but also some further optimizations / inlining.
   
   I think best would be to define a `HashMap` type alias, so we can use whatever is set by default across the project.


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org