Hello Adar Dembo,

I'd like you to do a code review.  Please visit

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

to review the following change.

Change subject: log_block_manager: reduce allocation at startup
......................................................................

log_block_manager: reduce allocation at startup

I found while restarting some servers that have a lot of blocks that
reading the block metadata files at startup got rather slow. In
particular, a profile revealed that the biggest CPU consumer was
allocation from deserializing BlockRecordPBs.

This changes the reading of the metadata files so that it reuses a
single BlockRecordPB instance and thus avoids allocation.

Change-Id: I441a00df6eb3ff10f21b45909f461698c79370ce
---
M src/kudu/fs/log_block_manager.cc
1 file changed, 25 insertions(+), 24 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/92/4192/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4192
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I441a00df6eb3ff10f21b45909f461698c79370ce
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>

Reply via email to