Hello Kudu Jenkins,

I'd like you to reexamine a change.  Please visit

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

to look at the new patch set (#2).

Change subject: KUDU-1692: tie various reader memtrackers to the tablet tracker
......................................................................

KUDU-1692: tie various reader memtrackers to the tablet tracker

Without the explicit connection, these memtrackers are all parented to the
root tracker, whose Release() behavior is rather expensive.

There's also some refactoring here:
- It was frustrating to pass the tablet's main and DMS memtrackers into
  various places, so there's a TabletMemTrackers struct to encapsulate them.
- There's some pass-by-value and std::move() for objects that are either
  large or may be large one day (e.g. ReaderOptions).
- CFileSet, MemRowSet and DeltaMemStore now follow the object construction
  idiom that we use elsewhere: a static method that creates the object and
  may return failure.

Note: this was originally https://gerrit.sjc.cloudera.com/#/c/7029. It's
since been rebased and all of the additional tracking was dropped.

Change-Id: I40ea59dc5d70c8ec935f9d96bcdb914c1d23ec5a
---
M src/kudu/cfile/bloomfile.cc
M src/kudu/cfile/bloomfile.h
M src/kudu/cfile/cfile_reader.cc
M src/kudu/cfile/cfile_reader.h
M src/kudu/tablet/cfile_set-test.cc
M src/kudu/tablet/cfile_set.cc
M src/kudu/tablet/cfile_set.h
M src/kudu/tablet/compaction-test.cc
M src/kudu/tablet/delta_compaction-test.cc
M src/kudu/tablet/delta_compaction.h
M src/kudu/tablet/delta_tracker.cc
M src/kudu/tablet/delta_tracker.h
M src/kudu/tablet/deltafile-test.cc
M src/kudu/tablet/deltafile.cc
M src/kudu/tablet/deltafile.h
M src/kudu/tablet/deltamemstore-test.cc
M src/kudu/tablet/deltamemstore.cc
M src/kudu/tablet/deltamemstore.h
M src/kudu/tablet/diskrowset-test-base.h
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/diskrowset.h
M src/kudu/tablet/memrowset-test.cc
M src/kudu/tablet/memrowset.cc
M src/kudu/tablet/memrowset.h
M src/kudu/tablet/tablet-test.cc
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet.h
A src/kudu/tablet/tablet_mem_trackers.h
M src/kudu/tools/tool_action_local_replica.cc
29 files changed, 490 insertions(+), 278 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/08/4708/2
-- 
To view, visit http://gerrit.cloudera.org:8080/4708
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I40ea59dc5d70c8ec935f9d96bcdb914c1d23ec5a
Gerrit-PatchSet: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>

Reply via email to