Andrew Wong has uploaded this change for review. ( 
http://gerrit.cloudera.org:8080/11249


Change subject: WIP KUDU-2469 pt 2: fail replicas on CFile corruption
......................................................................

WIP KUDU-2469 pt 2: fail replicas on CFile corruption

wip: add tests

This patch adds handling for CFile corruption errors via the error
manager, pulling the tablet id of interest with the SCOPED_IO_CONTEXT.

The tablet id is plumbed down at various points that may incur CFile IO:
- when performing any maintenance op
- when opening the tablet to open the keys
- when beginning a scan

I opted to add a new CFILE ErrorHandlerType enum rather than using the
TABLET enum (which is currently a no-op and serves to serialize certain
failures, see error_manager.h for more), as updating the TABLET error
handling behavior would change the existing error handling behavior.

Change-Id: I63d541443bc68c83fd0ca6d51315143fee04d50f
---
M src/kudu/cfile/bloomfile.cc
M src/kudu/cfile/cfile_reader.cc
M src/kudu/cfile/cfile_reader.h
M src/kudu/fs/error_manager.cc
M src/kudu/fs/error_manager.h
M src/kudu/tablet/tablet.cc
M src/kudu/tablet/tablet_mm_ops.cc
M src/kudu/tablet/tablet_replica_mm_ops.cc
M src/kudu/tserver/tablet_server.cc
M src/kudu/tserver/tablet_service.cc
10 files changed, 125 insertions(+), 28 deletions(-)



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newchange
Gerrit-Change-Id: I63d541443bc68c83fd0ca6d51315143fee04d50f
Gerrit-Change-Number: 11249
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>

Reply via email to