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 (#5). 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/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 M src/kudu/tserver/ts_tablet_manager.cc 8 files changed, 134 insertions(+), 9 deletions(-) git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/54/7654/5 -- 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: 5 Gerrit-Owner: Andrew Wong <aw...@cloudera.com> Gerrit-Reviewer: Adar Dembo <a...@cloudera.com> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Tidy Bot