Alexey Serbin has submitted this change and it was merged. ( http://gerrit.cloudera.org:8080/16709 )
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 Reviewed-on: http://gerrit.cloudera.org:8080/16709 Tested-by: Kudu Jenkins Reviewed-by: Alexey Serbin <[email protected]> --- 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(-) Approvals: Kudu Jenkins: Verified Alexey Serbin: Looks good to me, approved -- 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: merged Gerrit-Change-Id: I3bfbe12bf13d0cc274ad266f62c5539f7da791b8 Gerrit-Change-Number: 16709 Gerrit-PatchSet: 5 Gerrit-Owner: wangning <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: wangning <[email protected]>
