[
https://issues.apache.org/jira/browse/KUDU-3139?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17125116#comment-17125116
]
Tim Armstrong commented on KUDU-3139:
-------------------------------------
Todd suggested on IMPALA-9782 that my assumption that KuduPartitioner was
expensive to create was probably wrong, so we may not want to do this anyway...
> 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)
