Hello Tidy Bot, Kudu Jenkins, Adar Dembo,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/7654
to look at the new patch set (#2).
Change subject: handle disk failures during tablet copies
......................................................................
handle disk failures during tablet copies
There are two components in a tablet copy: the copy client (that
receives data) and the copy session source (that sends data).
Coarse-grain handling of disk failures during tablet copies is done for
the tablet copy client as:
* Before starting a copy client, if no disks are available to place the
tablet, simply return (instead of failing a CHECK).
* Before downloading each WALs or block, check that the tablet is in a
healthy group.
And for the tablet copy session as:
* Before sending a block or log segment, check if the tablet has an
error.
Upon returning an error, the tablet copy client will shutdown the
replica, leaving it in a failed state.
A test is added to ensure that both copy clients and that source
sessions with failed disks will return errors to the copying client.
Change-Id: Ic18d93c218ea13f3086f420a4847cb5e29a47bc7
---
M src/kudu/tablet/tablet.h
M src/kudu/tserver/tablet_copy-test-base.h
M src/kudu/tserver/tablet_copy_client-test.cc
M src/kudu/tserver/tablet_copy_client.cc
M src/kudu/tserver/tablet_copy_client.h
M src/kudu/tserver/tablet_copy_service-test.cc
M src/kudu/tserver/tablet_copy_source_session.cc
M src/kudu/tserver/tablet_copy_source_session.h
8 files changed, 119 insertions(+), 8 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/7654/2
--
To view, visit http://gerrit.cloudera.org:8080/7654
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: Ic18d93c218ea13f3086f420a4847cb5e29a47bc7
Gerrit-Change-Number: 7654
Gerrit-PatchSet: 2
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Tidy Bot
