Hello Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/16709
to look at the new patch set (#4).
Change subject: [tablet_copy_client-test] fix scenario on DownloadBlocks()
error reporting
......................................................................
[tablet_copy_client-test] fix scenario on DownloadBlocks() error reporting
In patch [1] the status reporting of the multi-threaded DownloadBlocks()
was not tested properly. The original test scenario called only
StartCopy(), but DownloadBlocks() was not called.
This patch moves the corresponding test code into a separate scenario,
targeting testing failure paths in any of the download threads spawned
from TabletCopyClient::DownloadBlocks(). With this patch,
both StartCopy() and DownloadBlocks() are called in the scenario.
[1]
https://gerrit.cloudera.org/#/c/16274/8/src/kudu/tserver/tablet_copy_client-test.cc
Change-Id: I3bfbe12bf13d0cc274ad266f62c5539f7da791b8
---
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
3 files changed, 23 insertions(+), 13 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/09/16709/4
--
To view, visit http://gerrit.cloudera.org:8080/16709
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I3bfbe12bf13d0cc274ad266f62c5539f7da791b8
Gerrit-Change-Number: 16709
Gerrit-PatchSet: 4
Gerrit-Owner: wangning <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: wangning <[email protected]>
