Hello Tidy Bot, Alexey Serbin, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17879
to look at the new patch set (#6).
Change subject: WIP KUDU-2671: Follow up pruning patch.
......................................................................
WIP KUDU-2671: Follow up pruning patch.
This patch modifies how partition key ranges are removed in
PartitionPruner::RemovePartitionKeyRange().
Currently, full scans using KuduScanner with no predicates are
functional. Scans with range predicates attached produce the right
result, but the order in which the tablets are returned from the
meta cache may prove to be problematic. They are returned by sorted
order of the hash component first, rather than the range. Further
testing is required to see if this manifests itself as an issue.
The current tests don't include tables with non covering ranges.
They also don't include scans with equality predicates (only
greater or less than predicates). They also do not include scans
using KuduScanTokens.
Change-Id: I3a1bf5344c0ef856072d3ed102714dce5ba21060
---
M src/kudu/client/client.h
M src/kudu/client/flex_partitioning_client-test.cc
M src/kudu/client/scan_token-internal.cc
M src/kudu/client/scan_token-test.cc
M src/kudu/client/scanner-internal.cc
M src/kudu/common/partition_pruner-test.cc
M src/kudu/common/partition_pruner.cc
M src/kudu/common/partition_pruner.h
8 files changed, 320 insertions(+), 71 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/79/17879/6
--
To view, visit http://gerrit.cloudera.org:8080/17879
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3a1bf5344c0ef856072d3ed102714dce5ba21060
Gerrit-Change-Number: 17879
Gerrit-PatchSet: 6
Gerrit-Owner: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <[email protected]>
Gerrit-Reviewer: Tidy Bot (241)
