Alexey Serbin has uploaded this change for review. ( http://gerrit.cloudera.org:8080/22762
Change subject: [tools] fix result status handling ...................................................................... [tools] fix result status handling This patch addresses sloppy result handling of functions/methods that return Status under the src/kudu/tools directory. It also includes a few fixes in other directories to address test failures due to updates in src/kudu/tools and unsorted follow-up fixes to prior patches on the result status handling. I also snuck in an update to address flakiness in the MaintenanceManagerTest.ComprehensiveTest scenario. Change-Id: I0d03b9b806d48a071777b63d497f049675d36a8f Reviewed-on: http://gerrit.cloudera.org:8080/22740 Tested-by: Alexey Serbin <[email protected]> Reviewed-by: Abhishek Chennaka <[email protected]> (cherry picked from commit 5ec9e470951c88d034e5e8d1c113923ada822128) --- M src/kudu/fs/log_block_manager-test.cc M src/kudu/integration-tests/create-table-itest.cc M src/kudu/integration-tests/delete_table-itest.cc M src/kudu/integration-tests/master_cert_authority-itest.cc M src/kudu/master/auto_rebalancer-test.cc M src/kudu/master/dynamic_multi_master-test.cc M src/kudu/tablet/delta_tracker.cc M src/kudu/tools/diagnostics_log_parser-test.cc M src/kudu/tools/ksck-test.cc M src/kudu/tools/ksck_remote-test.cc M src/kudu/tools/ksck_remote.cc M src/kudu/tools/ksck_results.cc M src/kudu/tools/kudu-admin-test.cc M src/kudu/tools/kudu-tool-test.cc M src/kudu/tools/master_rebuilder.cc M src/kudu/tools/table_scanner.cc M src/kudu/tools/tool_action-test.cc M src/kudu/tools/tool_action_fs.cc M src/kudu/tools/tool_action_hms.cc M src/kudu/tools/tool_action_local_replica.cc M src/kudu/tools/tool_action_master.cc M src/kudu/tools/tool_action_table.cc M src/kudu/tools/tool_action_txn.cc M src/kudu/transactions/txn_status_manager-test.cc M src/kudu/util/maintenance_manager-test.cc 25 files changed, 227 insertions(+), 173 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/62/22762/1 -- To view, visit http://gerrit.cloudera.org:8080/22762 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.18.x Gerrit-MessageType: newchange Gerrit-Change-Id: I0d03b9b806d48a071777b63d497f049675d36a8f Gerrit-Change-Number: 22762 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]>
