Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/22755 )
Change subject: [tserver] fix result status handling ...................................................................... [tserver] fix result status handling This patch addresses sloppy result handling of functions/methods that return Status under the src/kudu/tserver directory. Change-Id: If40eb0a273b13dc3b91817fc74a95a0dd99569a1 Reviewed-on: http://gerrit.cloudera.org:8080/22660 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Marton Greber <[email protected]> (cherry picked from commit 00619af3070a41b8eaba50290807e378a0285152) Reviewed-on: http://gerrit.cloudera.org:8080/22755 Reviewed-by: Abhishek Chennaka <[email protected]> --- M src/kudu/tserver/heartbeater.cc M src/kudu/tserver/tablet_copy_client-test.cc M src/kudu/tserver/tablet_copy_source_session-test.cc M src/kudu/tserver/tablet_server-test-base.cc M src/kudu/tserver/tablet_server-test-base.h M src/kudu/tserver/tablet_server-test.cc M src/kudu/tserver/tablet_server_authorization-test.cc M src/kudu/tserver/ts_tablet_manager.cc 8 files changed, 80 insertions(+), 53 deletions(-) Approvals: Alexey Serbin: Verified Abhishek Chennaka: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/22755 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: merged Gerrit-Change-Id: If40eb0a273b13dc3b91817fc74a95a0dd99569a1 Gerrit-Change-Number: 22755 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Marton Greber <[email protected]>
