Hello Dan Burkert, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/5276
to look at the new patch set (#3).
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
---
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(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/76/5276/3
--
To view, visit http://gerrit.cloudera.org:8080/5276
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I843cf483b93823cbcc5506958f62cbb076569ca8
Gerrit-PatchSet: 3
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]>
