You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@kudu.apache.org by "Tim Armstrong (Jira)" <ji...@apache.org> on 2020/06/02 17:58:00 UTC

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

Tim Armstrong created KUDU-3139:
-----------------------------------

             Summary: 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


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)