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 (#3).

Change subject: [test] fix tablet_copy_client-test not work for multi-thread 
DownloadBlocks status collect
......................................................................

[test] fix tablet_copy_client-test not work for multi-thread DownloadBlocks 
status collect

In patch
https://gerrit.cloudera.org/#/c/16274/8/src/kudu/tserver/tablet_copy_client-test.cc,
the error status collect cannot be test properly.
Currently the code aimed at testing whether parallelized version of 
DownloadBlocks()
can collect error status correctly or not.
However, it only called StartCopy() but no DownloadBlocks().

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, 22 insertions(+), 13 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/09/16709/3
--
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: 3
Gerrit-Owner: wangning <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: wangning <[email protected]>

Reply via email to