Hello Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/11249
to look at the new patch set (#2).
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 using the TABLET enum would
change the existing error handling behavior of certain errors.
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/2
--
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: newpatchset
Gerrit-Change-Id: I63d541443bc68c83fd0ca6d51315143fee04d50f
Gerrit-Change-Number: 11249
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
