Grant Henke has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/9193 )
Change subject: [tests] Add more test coverage for INT128 columns ...................................................................... [tests] Add more test coverage for INT128 columns Improves test coverage of reading and writing INT128 columns and fixes a predicate issue exposed by those tests. Also adds INT64 tests where it was missing. Change-Id: I3350634e55aad4316036c80d03e95e0acdf64295 Reviewed-on: http://gerrit.cloudera.org:8080/9193 Tested-by: Kudu Jenkins Reviewed-by: Dan Burkert <d...@cloudera.com> --- M src/kudu/cfile/cfile-test.cc M src/kudu/cfile/encoding-test.cc M src/kudu/client/scan_batch.cc M src/kudu/common/column_predicate.cc M src/kudu/common/partial_row.cc M src/kudu/tablet/all_types-scan-correctness-test.cc 6 files changed, 69 insertions(+), 11 deletions(-) Approvals: Kudu Jenkins: Verified Dan Burkert: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/9193 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I3350634e55aad4316036c80d03e95e0acdf64295 Gerrit-Change-Number: 9193 Gerrit-PatchSet: 10 Gerrit-Owner: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Dan Burkert <d...@cloudera.com> Gerrit-Reviewer: Grant Henke <granthe...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot