Hello Alexey Serbin, Andrew Wong,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/15447
to review the following change.
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
---
M src/kudu/benchmarks/tpch/tpch_real_world.cc
M src/kudu/benchmarks/wal_hiccup.cc
M src/kudu/cfile/bloomfile-test-base.h
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
49 files changed, 597 insertions(+), 778 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/47/15447/1
--
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: newchange
Gerrit-Change-Id: I39b87fc3d8542143507edf79a5ca05a7fc27ffd4
Gerrit-Change-Number: 15447
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
