Dan Burkert has submitted this change and it was merged. Change subject: Add internal IS NOT NULL predicate type ......................................................................
Add internal IS NOT NULL predicate type This commit adds an internal IS NOT NULL predicate type, and changes LESS_EQUAL predicate simplification to result in the IS NOT NULL predicate type when the conversion to a LESS predicate fails and the column is nullable. This fixes scans with predicates such as 'WHERE my_nullable_int8_col <= 127`. A followup commit will contain comprehensive predicate tests covering this case. Change-Id: Ifcf29b1f274df2ef5c5ac7a7a17cc06dfd59e191 Reviewed-on: http://gerrit.cloudera.org:8080/2671 Reviewed-by: Todd Lipcon <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/client/scan_predicate.cc M src/kudu/client/scanner-internal.cc M src/kudu/common/column_predicate-test.cc M src/kudu/common/column_predicate.cc M src/kudu/common/column_predicate.h M src/kudu/common/common.proto M src/kudu/tserver/tablet_service.cc 7 files changed, 120 insertions(+), 8 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2671 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ifcf29b1f274df2ef5c5ac7a7a17cc06dfd59e191 Gerrit-PatchSet: 5 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Dan Burkert <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Jean-Daniel Cryans Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
