Alexey Serbin has submitted this change and it was merged. Change subject: [tests] clean-up on kudu::client::ScanToStrings ......................................................................
[tests] clean-up on kudu::client::ScanToStrings With this patch, kudu::client::ScanToStrings() returns Status. The new signature is more convenient for localizing scan errors, if any. Change-Id: I8e7ba56d2e946c1ac9a4db2ef3a542f0a118471d Reviewed-on: http://gerrit.cloudera.org:8080/7027 Reviewed-by: Todd Lipcon <t...@apache.org> Tested-by: Kudu Jenkins --- M src/kudu/client/client-test-util.cc M src/kudu/client/client-test-util.h M src/kudu/client/client-test.cc M src/kudu/integration-tests/alter_table-randomized-test.cc M src/kudu/integration-tests/alter_table-test.cc M src/kudu/integration-tests/client-stress-test.cc M src/kudu/integration-tests/flex_partitioning-itest.cc 7 files changed, 41 insertions(+), 41 deletions(-) Approvals: Todd Lipcon: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/7027 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I8e7ba56d2e946c1ac9a4db2ef3a542f0a118471d Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <t...@apache.org>