Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/17764
Change subject: [common] more generic API for IN() predicate's pruning ...................................................................... [common] more generic API for IN() predicate's pruning While working on KUDU-2671, I found that the exposed internals of the PartitionSchema class doesn't allow for updating the implementation of the partition-related code to include per-range custom hash bucket schemas in a consistent manner. This patch introduces a bit more generic interface for pruning values of IN predicates by adding a new PartitionMayContainRow() method and removes the following methods from the public API of the PartitionSchema class: * HashPartitionContainsRow() * RangePartitionContainsRow() * IsColumnSingleRangeSchema() * TryGetSingleColumnHashPartitionIndex() I also added one extra test scenario and updated existing ones to allow for easier reading the assertion messages in case if those assertions ever trigger. This is a follow-up to 6a7cadc7e and 83b8caf4f. Change-Id: I2e2390cc4747864fdac71656dd7125ac3b15bf9d --- M src/kudu/common/partition.cc M src/kudu/common/partition.h M src/kudu/common/scan_spec-test.cc M src/kudu/common/scan_spec.cc M src/kudu/common/scan_spec.h 5 files changed, 417 insertions(+), 550 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/64/17764/1 -- To view, visit http://gerrit.cloudera.org:8080/17764 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: newchange Gerrit-Change-Id: I2e2390cc4747864fdac71656dd7125ac3b15bf9d Gerrit-Change-Number: 17764 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
