Hello Alexey Serbin, Kudu Jenkins, helifu, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/13858
to look at the new patch set (#2).
Change subject: KUDU-2635: treat failure to delete failed blocks as OK
......................................................................
KUDU-2635: treat failure to delete failed blocks as OK
It was previously possible to leave some orphaned blocks in the
in-memory orphaned blocks list by failing to delete some blocks due to a
disk failure. Upon deleting a tablet with such orphaned blocks, Kudu
could crash.
This patch addresses this by treating such failures to delete blocks
from failed data directories as OK, from the stance of actually deleting
the block. For all intents and purposes, the blocks aren't coming back.
A test is added that passed 1/100 times when run on dist-test in debug
mode. With the patch, it passes 100/100 times.
Change-Id: Ice78f41d6d367d42ad31c2127ceb5fc57a244e34
---
M src/kudu/fs/log_block_manager.cc
M src/kudu/tserver/tablet_server-test.cc
2 files changed, 38 insertions(+), 7 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/58/13858/2
--
To view, visit http://gerrit.cloudera.org:8080/13858
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ice78f41d6d367d42ad31c2127ceb5fc57a244e34
Gerrit-Change-Number: 13858
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: helifu <[email protected]>
