Dan Burkert has submitted this change and it was merged. Change subject: [c++ client] add comprehensive scan predicate tests ......................................................................
[c++ client] add comprehensive scan predicate tests Previously we had very little test coverage on predicates across column types. There is unfortunately a lot of code duplication between the test cases, but this is unavoidable because we don't have a way to generically set column values in a PartialRow or create a predicate. Change-Id: Id4a4c05a41a1e7984f330fe6ff912f3f9924c3a7 Reviewed-on: http://gerrit.cloudera.org:8080/2677 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/client/CMakeLists.txt A src/kudu/client/predicate-test.cc 2 files changed, 733 insertions(+), 0 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/2677 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Id4a4c05a41a1e7984f330fe6ff912f3f9924c3a7 Gerrit-PatchSet: 4 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: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
