[jira] [Resolved] (KUDU-3139) Add thread-safe KuduPartitioner::PartitionRow() method

["Tim Armstrong (Jira)" <ji...@apache.org>]

     [ https://issues.apache.org/jira/browse/KUDU-3139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Tim Armstrong resolved KUDU-3139.
---------------------------------
    Fix Version/s: NA
       Resolution: Not A Problem

We didn't end up needing this in Impala, so I'll close it.

> Add thread-safe KuduPartitioner::PartitionRow() method
> ------------------------------------------------------
>
>                 Key: KUDU-3139
>                 URL: https://issues.apache.org/jira/browse/KUDU-3139
>             Project: Kudu
>          Issue Type: Improvement
>          Components: client
>            Reporter: Tim Armstrong
>            Priority: Major
>              Labels: impala
>             Fix For: NA
>
>
> We have a scenario in IMPALA-9782 where we'd like to have multiple threads use KuduPartitioner, but this currently requires instantiating many instances, which is somewhat expensive. It would be a good improvement if we could share the KuduPartitioner. This currently doesn't work because PartitionRow() internally uses a buffer that's a member of the object - https://github.com/cloudera/kudu/blob/master/src/kudu/client/partitioner-internal.cc#L40
> I talked with [~aserbin] on slack and he suggested that adding a new method that is not stateful would be a good way to solve this (I also considered cloning KuduPartitioner).



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