You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2020/11/13 15:57:22 UTC

[GitHub] [arrow] Dandandan commented on pull request #8654: ARROW-10572: [Rust][Datafusion] Use aHash instead of FnvHashMap

Dandandan commented on pull request #8654:
URL: https://github.com/apache/arrow/pull/8654#issuecomment-726843795


   A nice overview is listed here: https://github.com/tkaitchuck/aHash/blob/master/compare/readme.md comparing aHash to other algorithms. aHash passes the full suit of https://github.com/rurban/smhasher , FnvHash has some weak points.
   
    @andygrove any way to test it? I recently also used it for the polars project where it brought around 20-30% speed ups to group by and hash join vs SeaHasher: https://github.com/ritchie46/polars/pull/128
   


----------------------------------------------------------------
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.

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