[ 
https://issues.apache.org/jira/browse/ARROW-1942?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16358798#comment-16358798
 ] 

ASF GitHub Bot commented on ARROW-1942:
---------------------------------------

xuepanchen commented on issue #1551: ARROW-1942: [C++] Hash table 
specializations for small integers
URL: https://github.com/apache/arrow/pull/1551#issuecomment-364520515
 
 
   @wesm got you. Compute-benchmark test failed when I run it locally and I 
can't figure out why. It seems to fail too before I added the 8-bit integer case

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


> [C++] Hash table specializations for small integers
> ---------------------------------------------------
>
>                 Key: ARROW-1942
>                 URL: https://issues.apache.org/jira/browse/ARROW-1942
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Wes McKinney
>            Assignee: Panchen Xue
>            Priority: Major
>              Labels: pull-request-available
>             Fix For: 0.9.0
>
>
> There is no need to use a dynamically-sized hash table with uint8, int8, 
> since a fixed-size lookup table can be used and avoid hashing altogether



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)

Reply via email to