Hello Kudu Jenkins, Andrew Wong, Hao Hao,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/17036
to look at the new patch set (#2).
Change subject: [util] remove AutoReleasePool and cleanup related code
......................................................................
[util] remove AutoReleasePool and cleanup related code
This patch removes AutoReleasePool class and changes just two places
where it was used to store heap-allocated objects. With this patch,
objects of appropriate type are wrapped into std::unique_ptr and
stored in std::deque container. Along the way, I did other minor
code cleanup.
The motivation for this change is reducing number of memory allocations
and removing unneeded complexity in the code.
Change-Id: I8adb5d08ec37e716baf524cb6c5f52366553883c
---
M src/kudu/client/client.cc
M src/kudu/client/scan_configuration.cc
M src/kudu/client/scan_configuration.h
M src/kudu/tablet/ops/alter_schema_op.cc
M src/kudu/tablet/ops/alter_schema_op.h
M src/kudu/tablet/ops/op.cc
M src/kudu/tablet/ops/op.h
D src/kudu/util/auto_release_pool.h
8 files changed, 49 insertions(+), 146 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/36/17036/2
--
To view, visit http://gerrit.cloudera.org:8080/17036
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8adb5d08ec37e716baf524cb6c5f52366553883c
Gerrit-Change-Number: 17036
Gerrit-PatchSet: 2
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Hao Hao <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
