Alexey Serbin has uploaded this change for review. (
http://gerrit.cloudera.org:8080/20702
Change subject: [tablet] modernize MultiColumnWriter's code
......................................................................
[tablet] modernize MultiColumnWriter's code
This patch contain a few micro-optimizations, but it doesn't contain
any functional changes:
* std::unique_ptr replaces raw pointers in the cfile_writers_ array,
so no need to use the STLDeleteElements macro
* replace CHECK() with DCHECK() where apprpriate
* removed virtual base from the class and made it final
* add tablet identifier into status objects and error messages
Change-Id: Ife716bf62338ca896072ce7fce3aea9d5f5204ca
---
M src/kudu/tablet/multi_column_writer.cc
M src/kudu/tablet/multi_column_writer.h
2 files changed, 56 insertions(+), 62 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/02/20702/1
--
To view, visit http://gerrit.cloudera.org:8080/20702
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: Ife716bf62338ca896072ce7fce3aea9d5f5204ca
Gerrit-Change-Number: 20702
Gerrit-PatchSet: 1
Gerrit-Owner: Alexey Serbin <[email protected]>
