You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kudu.apache.org by "Grant Henke (Jira)" <ji...@apache.org> on 2020/06/03 16:23:00 UTC
[jira] [Updated] (KUDU-3139) Add thread-safe
KuduPartitioner::PartitionRow() method
[ https://issues.apache.org/jira/browse/KUDU-3139?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]
Grant Henke updated KUDU-3139:
------------------------------
Labels: impala (was: )
> 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
>
> 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)