Todd Lipcon has submitted this change and it was merged. Change subject: KUDU-668. log block manager should tolerate empty metadata ......................................................................
KUDU-668. log block manager should tolerate empty metadata This solves a commonly seen issue after a server crashes due to "Too many open files". In many cases, this type of crash would leave an empty metadata file and a missing data file. This isn't a full fix for KUDU-668, but handles the above common case by just checking the size and existence of these files. If the metadata is too small to be valid, and the data is empty, then we just remove the container and continue with startup. Change-Id: Ie4fd9d0f510835d100d35efaf541d34829118f56 Reviewed-on: http://gerrit.cloudera.org:8080/4178 Reviewed-by: Adar Dembo <[email protected]> Tested-by: Kudu Jenkins --- M src/kudu/fs/block_manager-test.cc M src/kudu/fs/log_block_manager.cc M src/kudu/util/pb_util.cc M src/kudu/util/pb_util.h 4 files changed, 69 insertions(+), 12 deletions(-) Approvals: Adar Dembo: Looks good to me, approved Kudu Jenkins: Verified -- To view, visit http://gerrit.cloudera.org:8080/4178 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: merged Gerrit-Change-Id: Ie4fd9d0f510835d100d35efaf541d34829118f56 Gerrit-PatchSet: 3 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Todd Lipcon <[email protected]>
