Dan Burkert has submitted this change and it was merged. Change subject: [java client] KUDU-1643 Prune hash partitions based on IN-list predicates ......................................................................
[java client] KUDU-1643 Prune hash partitions based on IN-list predicates PartitionPruner is updated to search all combinations of in-list column values and prune hash partitions where possible. This also fixes a small issue in the C++ version of the algorithm: previously the implementation would always consider the final hash component to be constrained. The bug (as far as I can tell) doesn't lead to erroneous results, but does cause more partition ranges to be created, which results in more memory overhead and compute overhead to compute the pruning. Change-Id: I8a793c23ff00d19b3d3d062bb222d2c725a93724 Reviewed-on: http://gerrit.cloudera.org:8080/5677 Tested-by: Kudu Jenkins Reviewed-by: Jean-Daniel Cryans <[email protected]> --- M java/kudu-client/src/main/java/org/apache/kudu/client/PartitionPruner.java M java/kudu-client/src/test/java/org/apache/kudu/client/TestPartitionPruner.java 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 M src/kudu/common/scan_spec.cc 7 files changed, 535 insertions(+), 339 deletions(-) Approvals: Jean-Daniel Cryans: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5677 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8a793c23ff00d19b3d3d062bb222d2c725a93724 Gerrit-PatchSet: 10 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Haijie Hong <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
