Mike Percy has uploaded a new patch set (#2). Change subject: log: Mark allocation finished even if allocation had an error ......................................................................
log: Mark allocation finished even if allocation had an error The problem this patch fixes is that if a disk preallocation fails we will enter a "stuck" state where we cannot preallocate a new segment. Errors to allocate or append should be propagated up. A test exercising this code path is in a follow-up patch related to disk reservations. Since this is a bug fix, it seemed cleaner to separate it out into its own commit. Change-Id: If22bf946a42d0ec32c35164acd9e6e6cef18dcc3 --- M src/kudu/consensus/log.cc 1 file changed, 7 insertions(+), 7 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/34/3234/2 -- To view, visit http://gerrit.cloudera.org:8080/3234 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: If22bf946a42d0ec32c35164acd9e6e6cef18dcc3 Gerrit-PatchSet: 2 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Mike Percy <mpe...@apache.org> Gerrit-Reviewer: Todd Lipcon <t...@apache.org>