You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "David Ribeiro Alves (Code Review)" <ge...@cloudera.org> on 2018/07/27 22:35:47 UTC

[kudu-CR] [tablet] Avoid compactions in the past

David Ribeiro Alves has posted comments on this change. ( http://gerrit.cloudera.org:8080/8965 )

Change subject: [tablet] Avoid compactions in the past
......................................................................


Patch Set 11:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/8965/11/src/kudu/tablet/tablet_bootstrap.cc
File src/kudu/tablet/tablet_bootstrap.cc:

http://gerrit.cloudera.org:8080/#/c/8965/11/src/kudu/tablet/tablet_bootstrap.cc@1116
PS11, Line 1116:   tablet_->mvcc_manager()->AdjustSafeTime(safe_time);
this needs to advance clean time too otherwise we'll still end up with an empty



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I6db9394d37b4ad1e7b1b1bc6ba0535558c20d9a8
Gerrit-Change-Number: 8965
Gerrit-PatchSet: 11
Gerrit-Owner: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: David Ribeiro Alves <da...@gmail.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
Gerrit-Reviewer: Todd Lipcon <to...@apache.org>
Gerrit-Comment-Date: Fri, 27 Jul 2018 22:35:47 +0000
Gerrit-HasComments: Yes