Hello Mike Percy,

I'd like you to do a code review.  Please visit

    http://gerrit.cloudera.org:8080/4319

to review the following change.

Change subject: log_util: downgrade scary "missing footer" message to INFO
......................................................................

log_util: downgrade scary "missing footer" message to INFO

In most cases if a Kudu server restarts, it will leave log segments with
no footer. This previously caused a scary-looking warning to be emitted
on the next startup.

This downgrades the message to INFO for the case of the footer not being
found. I also fixed this code path to properly propagate other types of
errors (eg IOError) which might occur while reading the footer.

Change-Id: Ic60f57e1276e6e4cc209d37d6fdad444c24db3c6
---
M src/kudu/consensus/log_util.cc
1 file changed, 10 insertions(+), 4 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/19/4319/1
-- 
To view, visit http://gerrit.cloudera.org:8080/4319
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: Ic60f57e1276e6e4cc209d37d6fdad444c24db3c6
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Mike Percy <mpe...@apache.org>

Reply via email to