Andrew Wong has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/15506 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/15506 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- 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, 147 insertions(+), 85 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- 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: merged Gerrit-Change-Id: Iea2f28fb2905ddcc007c88ab80ae2185587400f0 Gerrit-Change-Number: 15506 Gerrit-PatchSet: 8 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)
