Hello Alexey Serbin, Andrew Wong,
I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/15448
to review the following change.
Change subject: thread: simplify Thread::Create API
......................................................................
thread: simplify Thread::Create API
I set out to remove boost::bind from this API, but after that I found it
cleaner to force callers to create the lambdas rather than retain the
Thread::Create multi-arg overloads.
Change-Id: I581378b093a8d2e8536b7b07cca5520fdaa7672d
---
M src/kudu/clock/builtin_ntp.cc
M src/kudu/hms/hms_catalog.cc
M src/kudu/master/catalog_manager.cc
M src/kudu/master/hms_notification_log_listener.cc
M src/kudu/rpc/acceptor_pool.cc
M src/kudu/rpc/reactor.cc
M src/kudu/rpc/result_tracker.cc
M src/kudu/rpc/retriable_rpc.h
M src/kudu/rpc/service_pool.cc
M src/kudu/server/diagnostics_log.cc
M src/kudu/server/server_base.cc
M src/kudu/subprocess/server.cc
M src/kudu/tserver/heartbeater.cc
M src/kudu/tserver/scanners.cc
M src/kudu/tserver/tablet_copy_service.cc
M src/kudu/util/cloud/instance_detector.cc
M src/kudu/util/debug-util-test.cc
M src/kudu/util/file_cache.cc
M src/kudu/util/kernel_stack_watchdog.cc
M src/kudu/util/maintenance_manager.cc
M src/kudu/util/minidump.cc
M src/kudu/util/pstack_watcher.cc
M src/kudu/util/thread-test.cc
M src/kudu/util/thread.cc
M src/kudu/util/thread.h
M src/kudu/util/threadpool.cc
M src/kudu/util/trace-test.cc
M src/kudu/util/ttl_cache.h
28 files changed, 132 insertions(+), 171 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/15448/1
--
To view, visit http://gerrit.cloudera.org:8080/15448
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I581378b093a8d2e8536b7b07cca5520fdaa7672d
Gerrit-Change-Number: 15448
Gerrit-PatchSet: 1
Gerrit-Owner: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
