You are viewing a plain text version of this content. The canonical link for it is here.

Posted to reviews@kudu.apache.org by "Andrew Wong (Code Review)" <ge...@cloudera.org> on 2020/02/02 23:34:31 UTC

[kudu-CR] WIP KUDU-1625: background op to GC ancient, empty rowsets

Andrew Wong has posted comments on this change. ( http://gerrit.cloudera.org:8080/15145 )

Change subject: WIP KUDU-1625: background op to GC ancient, empty rowsets
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/15145/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/15145/1//COMMIT_MSG@19
PS1, Line 19: questions
            : around write amplification and compatibility with the existing
            : compactions strategies
> Is it possible to track the write activity on the tablet replica and make a
Yeah, I think that kind of solution would look somewhat similar to this patch -- the addition of a new maintenance op. Though it'd probably be a lower priority op as to not impede regular compactions. Contrast that to this op, which is relatively high priority since it's cheap to execute.



-- 
To view, visit http://gerrit.cloudera.org:8080/15145
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I696e2a29ea52ad4e54801b495c322bc371787124
Gerrit-Change-Number: 15145
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Adar Dembo <ad...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <as...@cloudera.com>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Comment-Date: Sun, 02 Feb 2020 23:34:31 +0000
Gerrit-HasComments: Yes