Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1446 Consider data types in predicate evaluation order ......................................................................
KUDU-1446 Consider data types in predicate evaluation order This patch enhances SelectivityRank() in column_predicate.cc to account for column type. Predicate type dominates column type for the sort, and a predicate on a column of a small type will sort before a predicate on a column of larger (or variable-size) type. It also adds a small test for selectivity comparison. Change-Id: I9c3388876035cb494fdb2c0e1f17698f089fdf72 Reviewed-on: http://gerrit.cloudera.org:8080/3298 Tested-by: Kudu Jenkins Reviewed-by: Todd Lipcon <t...@apache.org> --- M src/kudu/common/column_predicate-test.cc M src/kudu/common/column_predicate.cc 2 files changed, 49 insertions(+), 5 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/3298 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9c3388876035cb494fdb2c0e1f17698f089fdf72 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Will Berkeley <wdberke...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org> Gerrit-Reviewer: Will Berkeley <wdberke...@gmail.com>