Hello Tidy Bot, Kudu Jenkins, Adar Dembo,

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

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

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

Change subject: tablet: plumb delta stats into delta compaction outputs
......................................................................

tablet: plumb delta stats into delta compaction outputs

This plumbs delta stats from a delta compaction into the delta tracker's
call to open the output delta readers. This means that in-memory delta
stats aren't "lost" when doing a delta compaction, and that we can GC
ancient deleted rowsets after a delta compaction without reading the
stats from disk.

I considered also plumbing stats into merge compaction outputs, but
opted not to. The plumbing for compactions seems more significant, and
chances are, large bulk deletes will come after data has had time to be
compacted (e.g. deleting old ranges of keyspace).

Change-Id: Iea2f28fb2905ddcc007c88ab80ae2185587400f0
---
M src/kudu/integration-tests/tablet_history_gc-itest.cc
M src/kudu/tablet/delta_compaction-test.cc
M src/kudu/tablet/delta_compaction.cc
M src/kudu/tablet/delta_store.cc
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.cc
M src/kudu/tablet/tablet-test-util.h
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet_history_gc-test.cc
16 files changed, 145 insertions(+), 84 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/06/15506/5
--
To view, visit http://gerrit.cloudera.org:8080/15506
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Iea2f28fb2905ddcc007c88ab80ae2185587400f0
Gerrit-Change-Number: 15506
Gerrit-PatchSet: 5
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)

Reply via email to