[jira] [Commented] (ARROW-13706) [C++][Compute] Add Find method to Grouper

["Niranda Perera (Jira)" <ji...@apache.org>]

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

Niranda Perera commented on ARROW-13706:
----------------------------------------

There is a partial implementation this in ARROW-13268 (Adds Find only to GrouperImpl). I will add Find to GrouperFastImpl once ARROW-13532 is merged.

> [C++][Compute] Add Find method to Grouper
> -----------------------------------------
>
>                 Key: ARROW-13706
>                 URL: https://issues.apache.org/jira/browse/ARROW-13706
>             Project: Apache Arrow
>          Issue Type: Improvement
>          Components: C++
>            Reporter: Michal Nowakiewicz
>            Priority: Major
>              Labels: query-engine
>             Fix For: 6.0.0
>
>
> After SwissTable implements find() method in ARROW-13532, Grouper can add Find() method to its interface. Note that the main problem is that Grouper uses mutable member variables for storing temporary data, whenever it is doing hash table lookups, even when the hash table is used as read-only. Grouper is also not thread-safe, while Find() method ideally should be thread-safe, working on immutable object.



--
This message was sent by Atlassian Jira
(v8.3.4#803005)