Hello Kudu Jenkins, Adar Dembo,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/14555

to look at the new patch set (#2).

Change subject: KUDU-2977 sharding block map to speed up tserver startup
......................................................................

KUDU-2977 sharding block map to speed up tserver startup

Separate LogBlockManager's block lock and container lock,
and sharding block map.

Time cost of LogBlockManagerTest.StartupBenchmark in old version:
I1026 17:17:30.231856 150030 log_block_manager-test.cc:1042] Time spent 
reopening block manager: real 1.148s    user 0.092s    sys 0.030s
I1026 17:17:30.798593 160153 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.062s    user 0.062s    sys 0.000s
I1026 17:17:30.866621 160151 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.109s    user 0.068s    sys 0.000s
I1026 17:17:30.949687 160154 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.192s    user 0.083s    sys 0.000s
I1026 17:17:31.007802 160147 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.264s    user 0.058s    sys 0.000s
I1026 17:17:31.103804 160149 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.339s    user 0.094s    sys 0.002s
I1026 17:17:31.171495 160150 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.417s    user 0.068s    sys 0.000s
I1026 17:17:31.252955 160148 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.502s    user 0.081s    sys 0.000s
I1026 17:17:31.331007 160152 log_block_manager.cc:2588] Time spent 
AddLogBlocks...: real 0.581s    user 0.078s    sys 0.000s

new version:
I1026 17:15:48.425242 52360 log_block_manager-test.cc:1042] Time spent 
reopening block manager: real 0.945s    user 0.159s    sys 0.001s
I1026 17:15:49.245585 64003 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.256s    user 0.162s    sys 0.022s
I1026 17:15:49.323187 63997 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.315s    user 0.190s    sys 0.027s
I1026 17:15:49.351920 64004 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.330s    user 0.213s    sys 0.042s
I1026 17:15:49.377976 63999 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.341s    user 0.229s    sys 0.032s
I1026 17:15:49.380051 64002 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.336s    user 0.208s    sys 0.034s
I1026 17:15:49.387194 63998 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.330s    user 0.215s    sys 0.040s
I1026 17:15:49.399719 64001 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.317s    user 0.206s    sys 0.034s
I1026 17:15:49.434571 64000 log_block_manager.cc:2607] Time spent 
AddLogBlock...: real 0.322s    user 0.187s    sys 0.061s

Change-Id: If0d5c13e051a2c1d6cfd1c9ad7db8a3cd195459d
---
M src/kudu/fs/log_block_manager-test.cc
M src/kudu/fs/log_block_manager.cc
M src/kudu/fs/log_block_manager.h
3 files changed, 156 insertions(+), 103 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/55/14555/2
--
To view, visit http://gerrit.cloudera.org:8080/14555
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: If0d5c13e051a2c1d6cfd1c9ad7db8a3cd195459d
Gerrit-Change-Number: 14555
Gerrit-PatchSet: 2
Gerrit-Owner: Yingchun Lai <405403...@qq.com>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to