Hello Yingchun Lai, Kudu Jenkins, Abhishek Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21543
to look at the new patch set (#2).
Change subject: [cfile] allocate CFileWriter field on the stack when possible
......................................................................
[cfile] allocate CFileWriter field on the stack when possible
This patch updates BloomFileWriter and DeltaFileWriter classes to avoid
allocating their CFileWriter member field on the heap, and also contains
other minor clean-up. It makes the code cleaner and results in less
calls to new/tcmalloc.
This patch doesn't contain any functional modifications.
Change-Id: I910c8dfac88cb1018d76ead20034fdae355a1d4c
---
M src/kudu/cfile/bloomfile.cc
M src/kudu/cfile/bloomfile.h
M src/kudu/cfile/cfile_util.cc
M src/kudu/cfile/cfile_util.h
M src/kudu/cfile/cfile_writer.cc
M src/kudu/cfile/cfile_writer.h
M src/kudu/common/schema.h
M src/kudu/tablet/deltafile.cc
M src/kudu/tablet/deltafile.h
M src/kudu/util/bloom_filter.h
10 files changed, 171 insertions(+), 80 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/43/21543/2
--
To view, visit http://gerrit.cloudera.org:8080/21543
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I910c8dfac88cb1018d76ead20034fdae355a1d4c
Gerrit-Change-Number: 21543
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Abhishek Chennaka <achenn...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yingchun Lai <laiyingc...@apache.org>
