Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Andrew Wong,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17643
to look at the new patch set (#2).
Change subject: [pruning] wip KUDU-2671: Modifies pruning logic to include per
range hash schemas.
......................................................................
[pruning] wip KUDU-2671: Modifies pruning logic to include per range hash
schemas.
The PartitionPruner class is responsible for constructing
partition key ranges from the predicates and PK bounds
provided by the scan. Currently, when the ranges are constructed,
the same table-wide set of hash bucket schemas is used for every range.
With this patch, that will no longer be the case as now ranges are
constructed using PartitionSchema's new field of per range hash schemas.
WIP patch because the functionality of pruning given partition keys
as part of the scan is incomplete. However, according to the client API,
this is unstable and for internal use only.
PartitionPruner::RemovePartitionKeyRange removes ranges using a partition key
so I'll leave this patch as a WIP for the time being as it is a critical
function.
Change-Id: I05c37495430f61a2c6f6012c72251138aee465b7
---
M src/kudu/common/partial_row.h
M src/kudu/common/partition.cc
M src/kudu/common/partition.h
M src/kudu/common/partition_pruner-test.cc
M src/kudu/common/partition_pruner.cc
M src/kudu/common/partition_pruner.h
6 files changed, 763 insertions(+), 163 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/43/17643/2
--
To view, visit http://gerrit.cloudera.org:8080/17643
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I05c37495430f61a2c6f6012c72251138aee465b7
Gerrit-Change-Number: 17643
Gerrit-PatchSet: 2
Gerrit-Owner: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Tidy Bot (241)
