Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/18053 )

Change subject: [bootstrap] Speedup tablet bootstrap
......................................................................

[bootstrap] Speedup tablet bootstrap

Speedup tablet bootstrap by
- using multi-threads to load tablets metadata
- short circuit return when found an active memory store
  in TabletBootstrap::AnalyzeActiveStores()
- improving branch prediction by adding more PREDICT_FALSE/TRUE
  on hot path

A simple startup benchmark has been added, it shows that this patch can
reduce about 85% tablets metadata load time when set
--num_tablets_to_open_simultaneously from default 1 to 8

Change-Id: I816417d2d4c24014edb6b2a40c060f29e37ae219
Reviewed-on: http://gerrit.cloudera.org:8080/18053
Tested-by: Kudu Jenkins
Reviewed-by: Alexey Serbin <[email protected]>
Reviewed-by: Attila Bukor <[email protected]>
---
M src/kudu/fs/fs_manager.cc
M src/kudu/fs/log_block_manager.cc
M src/kudu/tablet/tablet_bootstrap.cc
M src/kudu/tablet/tablet_metadata.cc
M src/kudu/tserver/ts_tablet_manager-test.cc
M src/kudu/tserver/ts_tablet_manager.cc
M src/kudu/tserver/ts_tablet_manager.h
7 files changed, 183 insertions(+), 91 deletions(-)

Approvals:
  Kudu Jenkins: Verified
  Alexey Serbin: Looks good to me, approved
  Attila Bukor: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/18053
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I816417d2d4c24014edb6b2a40c060f29e37ae219
Gerrit-Change-Number: 18053
Gerrit-PatchSet: 22
Gerrit-Owner: Yingchun Lai <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Yingchun Lai <[email protected]>

Reply via email to