Adar Dembo has posted comments on this change. Change subject: KUDU-236. Add tablet history GC design doc ......................................................................
Patch Set 2: (2 comments) Just passing through... http://gerrit.cloudera.org:8080/#/c/2853/2/docs/design-docs/tablet-history-gc.md File docs/design-docs/tablet-history-gc.md: Line 60: with the explicit purpose ot removing old rows. This task will run a special How expensive is the Row GC task? If I'm understanding it correctly, it makes a copy of a DiskRowSet with N rows missing (the greater N, the higher the 'score'); is that right? Line 64: Defining the Row GC task will require a scoring function that takes into The score for Row GC will generally be lower than for merge compaction, right? Since the latter can also GC old rows? -- To view, visit http://gerrit.cloudera.org:8080/2853 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: Ibd1776971ac4afa15a34daeced052913cf887dc3 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dan Burkert <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: Yes
