Hello David Ribeiro Alves, Todd Lipcon, I'd like you to do a code review. Please visit
http://gerrit.cloudera.org:8080/9482 to review the following change. Change subject: Don't perform compactions when clean time has not been advanced ...................................................................... Don't perform compactions when clean time has not been advanced Due to KUDU-2233 we might not advance safe time, and thus clean time, at bootstrap causing possible corruption or crashes. This patch adds a check to make sure that clean time has been advanced at all before performing a compaction. This is temporary fix until we have a more strict check like the one proposed in https://gerrit.cloudera.org/#/c/8887/. Change-Id: Ia74abdf7d806efc4239dc9cff4a5da28621d331a Reviewed-on: http://gerrit.cloudera.org:8080/9436 Reviewed-by: Todd Lipcon <t...@apache.org> Tested-by: David Ribeiro Alves <davidral...@gmail.com> (cherry picked from commit 00815045fc3295c12023fd7ae7ad220645a10c3a) --- M src/kudu/tablet/tablet.cc 1 file changed, 7 insertions(+), 0 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/82/9482/1 -- To view, visit http://gerrit.cloudera.org:8080/9482 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: branch-1.6.x Gerrit-MessageType: newchange Gerrit-Change-Id: Ia74abdf7d806efc4239dc9cff4a5da28621d331a Gerrit-Change-Number: 9482 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <aser...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>