Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/11032 )
Change subject: [tablet] clean-up on replay of WAL entries ...................................................................... [tablet] clean-up on replay of WAL entries Simplified handling of the ownership of log entries read from disk. This changelist also contains other changes in tablet_bootstrap.cc and corresponding WAL utilities: * reduced number of memory allocations while reading WAL entries * replaced gscoped_ptr with std::unique_ptr * other minor updates around WAL-related utilities and tests Change-Id: Ie39516b09e4c756e8af1d8e1a5604672c96a80cb Reviewed-on: http://gerrit.cloudera.org:8080/11032 Tested-by: Alexey Serbin <aser...@cloudera.com> Reviewed-by: Adar Dembo <a...@cloudera.com> --- M src/kudu/consensus/log-test-base.h M src/kudu/consensus/log-test.cc M src/kudu/consensus/log_reader.cc M src/kudu/consensus/log_reader.h M src/kudu/consensus/log_util.cc M src/kudu/consensus/log_util.h M src/kudu/consensus/raft_consensus_quorum-test.cc M src/kudu/integration-tests/log_verifier.cc M src/kudu/tablet/tablet_bootstrap.cc M src/kudu/tools/tool_action_common.cc M src/kudu/tools/tool_action_local_replica.cc 11 files changed, 256 insertions(+), 239 deletions(-) Approvals: Alexey Serbin: Verified Adar Dembo: Looks good to me, approved -- To view, visit http://gerrit.cloudera.org:8080/11032 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: merged Gerrit-Change-Id: Ie39516b09e4c756e8af1d8e1a5604672c96a80cb Gerrit-Change-Number: 11032 Gerrit-PatchSet: 3 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>