wangning has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/16709 )

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


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/16709/1/src/kudu/tserver/tablet_copy_client-test.cc
File src/kudu/tserver/tablet_copy_client-test.cc:

http://gerrit.cloudera.org:8080/#/c/16709/1/src/kudu/tserver/tablet_copy_client-test.cc@a214
PS1, Line 214:
             :
             :
             :
             :
             :
             :
             :
             :
             :
             :
             :
             :
             :
Hello Alexey,
when re-reading the code of this test I wrote before (url is mentioned in 
commit-msg), I found the previously the code could not satisfy the test need 
"test the error status can be collected by multi-threads".
Sorry for troubling you.



--
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: comment
Gerrit-Change-Id: I3bfbe12bf13d0cc274ad266f62c5539f7da791b8
Gerrit-Change-Number: 16709
Gerrit-PatchSet: 1
Gerrit-Owner: wangning <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: wangning <[email protected]>
Gerrit-Comment-Date: Wed, 11 Nov 2020 06:23:35 +0000
Gerrit-HasComments: Yes

Reply via email to