Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/17643 )
Change subject: [pruning] KUDU-2671: Pruning compatible with custom hash schemas. ...................................................................... [pruning] KUDU-2671: Pruning compatible with custom hash schemas. This patch introduces changes to the PartitionPruner class to be compatible with custom hash bucket schemas per range. There are three ways to set bounds on a scan. - Adding predicates (e.g. range/equality) - Setting lower and upper bound primary keys - Setting lower and upper bound partition keys This patch introduces changes that make the first two methods of setting bounds on a scan compatible with custom hash bucket schemas per range. The last way using partition keys is unstable and for internal use only. While it's not necessary for the last way to be compatible with per range hash bucket schemas, the entire pruning functionality will not be complete until PartitionPruner::RemovePartitionKeyRange() is modified. That work will be done in a follow up patch. Change-Id: I05c37495430f61a2c6f6012c72251138aee465b7 Reviewed-on: http://gerrit.cloudera.org:8080/17643 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/common/partial_row.h M src/kudu/common/partition_pruner-test.cc M src/kudu/common/partition_pruner.cc M src/kudu/common/partition_pruner.h 4 files changed, 1,051 insertions(+), 389 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I05c37495430f61a2c6f6012c72251138aee465b7 Gerrit-Change-Number: 17643 Gerrit-PatchSet: 11 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)
