Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-1551. Ignore log segments which were preallocated but have no header. ......................................................................
KUDU-1551. Ignore log segments which were preallocated but have no header. This fixes a TS startup crash in the case that it finds a log segment which is preallocated but has no valid magic or header at the start. The new test injects such a crash and verifies that the server can restart and replay its logs. Change-Id: I843cf483b93823cbcc5506958f62cbb076569ca8 Reviewed-on: http://gerrit.cloudera.org:8080/5276 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/consensus/log-test.cc M src/kudu/consensus/log_reader.cc M src/kudu/consensus/log_util.cc M src/kudu/consensus/log_util.h M src/kudu/integration-tests/ts_recovery-itest.cc 5 files changed, 66 insertions(+), 14 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/5276 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: I843cf483b93823cbcc5506958f62cbb076569ca8 Gerrit-PatchSet: 4 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[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: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]>
