Adar Dembo has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15447 )
Change subject: remove kudu::Thread from tests ...................................................................... remove kudu::Thread from tests std::thread is much more ergonomic, evidenced in part by the silly kudu::Thread names and categories found in tests. I retained kudu::Thread in debug-util-test and trace-test, both of which rely on some internal Kudu threading detail. And of course in thread-test which actually exercises this functionality. Change-Id: I39b87fc3d8542143507edf79a5ca05a7fc27ffd4 Reviewed-on: http://gerrit.cloudera.org:8080/15447 Reviewed-by: Alexey Serbin <[email protected]> Tested-by: Kudu Jenkins Reviewed-by: Andrew Wong <[email protected]> --- M src/kudu/benchmarks/tpch/tpch_real_world.cc M src/kudu/benchmarks/wal_hiccup.cc M src/kudu/cfile/mt-bloomfile-test.cc M src/kudu/client/client-test.cc M src/kudu/clock/hybrid_clock-test.cc M src/kudu/consensus/mt-log-test.cc M src/kudu/fs/block_manager-stress-test.cc M src/kudu/fs/block_manager-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/consistency-itest.cc M src/kudu/integration-tests/create-table-itest.cc M src/kudu/integration-tests/exactly_once_writes-itest.cc M src/kudu/integration-tests/full_stack-insert-scan-test.cc M src/kudu/integration-tests/linked_list-test-util.h M src/kudu/integration-tests/master_replication-itest.cc M src/kudu/integration-tests/raft_consensus-itest-base.cc M src/kudu/integration-tests/raft_consensus-itest-base.h M src/kudu/integration-tests/raft_consensus-itest.cc M src/kudu/integration-tests/raft_consensus_election-itest.cc M src/kudu/integration-tests/tablet_copy_client_session-itest.cc M src/kudu/integration-tests/ts_recovery-itest.cc M src/kudu/integration-tests/update_scan_delta_compact-test.cc M src/kudu/master/hms_notification_log_listener.cc M src/kudu/mini-cluster/webui_checker.cc M src/kudu/mini-cluster/webui_checker.h M src/kudu/rpc/exactly_once_rpc-test.cc M src/kudu/rpc/mt-rpc-test.cc M src/kudu/rpc/rpc-test-base.h M src/kudu/rpc/rpc-test.cc M src/kudu/tablet/lock_manager-test.cc M src/kudu/tablet/mt-rowset_delta_compaction-test.cc M src/kudu/tablet/mt-tablet-test.cc M src/kudu/tablet/tablet_random_access-test.cc M src/kudu/tablet/transactions/transaction_tracker-test.cc M src/kudu/tools/ksck_remote-test.cc M src/kudu/tserver/tablet_server-stress-test.cc M src/kudu/tserver/tablet_server-test.cc M src/kudu/util/countdown_latch-test.cc M src/kudu/util/maintenance_manager-test.cc M src/kudu/util/mt-hdr_histogram-test.cc M src/kudu/util/mt-metrics-test.cc M src/kudu/util/mt-threadlocal-test.cc M src/kudu/util/once-test.cc M src/kudu/util/pstack_watcher.cc M src/kudu/util/striped64-test.cc M src/kudu/util/test_graph.cc M src/kudu/util/test_graph.h 48 files changed, 618 insertions(+), 792 deletions(-) Approvals: Alexey Serbin: Looks good to me, approved Kudu Jenkins: Verified Andrew Wong: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/15447 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: I39b87fc3d8542143507edf79a5ca05a7fc27ffd4 Gerrit-Change-Number: 15447 Gerrit-PatchSet: 5 Gerrit-Owner: Adar Dembo <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120)
