Hello Kudu Jenkins, I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7030 to look at the new patch set (#17). Change subject: disk failure: error-handling macros in blocks ...................................................................... disk failure: error-handling macros in blocks This patch adds macros to trigger error-handling code in the event of disk failure during block IO. Block- and container-level functions that call env or cache functions (e.g. File::Append, FileCache::DeleteFile) that may result in disk failure can now run callbacks that, in the future, will fail and shutdown tablets in the failed directory, preventing further IO to the disk. Disk failures are indicated by one of EIO, ENODEV, ENXIO, and EROFS, although this list can be extended as needed in the future. As failure-handling relies on the addition of a handful of other features, the meat of the error-handling callback will come in a later patch. This patch only serves to surround block IO calls with macros. Failures elswhere (e.g. when reading instance or tablet metadata) will be handled in separate patches. This is part of a series of patches to handle disk failure. See section 2.2 of: https://docs.google.com/document/d/1zZk-vb_ETKUuePcZ9ZqoSK2oPvAAaEV1sjDXes8Pxgk/edit?usp=sharing Change-Id: Ia03bfb711a1b022d7516f4adb37fe9fb28ec949c --- M src/kudu/fs/error_manager.h M src/kudu/fs/file_block_manager.cc M src/kudu/fs/log_block_manager.cc M src/kudu/tserver/ts_tablet_manager.cc M src/kudu/util/status.h 5 files changed, 191 insertions(+), 67 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/30/7030/17 -- To view, visit http://gerrit.cloudera.org:8080/7030 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: newpatchset Gerrit-Change-Id: Ia03bfb711a1b022d7516f4adb37fe9fb28ec949c Gerrit-PatchSet: 17 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: David Ribeiro Alves <davidral...@gmail.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <t...@apache.org>