You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@kudu.apache.org by "Mike Percy (Code Review)" <ge...@cloudera.org> on 2016/05/16 01:23:06 UTC

[kudu-CR] WIP: KUDU-236. Implement tablet history GC

Mike Percy has uploaded a new change for review.

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

Change subject: WIP: KUDU-236. Implement tablet history GC
......................................................................

WIP: KUDU-236. Implement tablet history GC

This is a draft of tablet history GC. Test coverage is not yet complete.

Tests:

* A positive test for entire-row GC.

Missing tests:

* Missing an undo GC test (!)
* Missing negative test to ensure we don't GC rows more often than we should
* Missing randomized tests

Missing features:

* No mechanism to ensure history is not GCed while open scanners are
  iterating on a timestamp.
* No dedicated background task that only schedules and performs history GC.

Change-Id: If9833a863f118eb82be80ea56204d0d9141611c2
---
M src/kudu/server/clock.h
M src/kudu/server/hybrid_clock.cc
M src/kudu/server/hybrid_clock.h
M src/kudu/tablet/CMakeLists.txt
M src/kudu/tablet/compaction-test.cc
M src/kudu/tablet/compaction.cc
M src/kudu/tablet/compaction.h
M src/kudu/tablet/delta_compaction.cc
M src/kudu/tablet/delta_compaction.h
M src/kudu/tablet/diskrowset.cc
M src/kudu/tablet/diskrowset.h
M src/kudu/tablet/mutation.h
M src/kudu/tablet/rowset_metadata.h
M src/kudu/tablet/tablet-harness.h
M src/kudu/tablet/tablet-test-base.h
M src/kudu/tablet/tablet-test-util.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_history_gc-test.cc
20 files changed, 319 insertions(+), 67 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/3076/1
-- 
To view, visit http://gerrit.cloudera.org:8080/3076
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: If9833a863f118eb82be80ea56204d0d9141611c2
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <mp...@apache.org>