Adar Dembo has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/15028 )

Change subject: tablet_copy: fail tablet when concurrent IO fails copy
......................................................................


Patch Set 1: Code-Review+2

(3 comments)

Good fix. This test has been flaky for over 1.5 years and it's due to a real 
product bug. I wonder if users have been seeing it in the wild?

http://gerrit.cloudera.org:8080/#/c/15028/1//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/15028/1//COMMIT_MSG@7
PS1, Line 7: tablet_copy: fail tablet when concurrent IO fails copy
Tag with KUDU-2496?


http://gerrit.cloudera.org:8080/#/c/15028/1//COMMIT_MSG@21
PS1, Line 21: the
Nit: drop


http://gerrit.cloudera.org:8080/#/c/15028/1//COMMIT_MSG@22
PS1, Line 22: no
not



--
To view, visit http://gerrit.cloudera.org:8080/15028
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ie270f435174fb8fba2adea21a5fbb48f3e56e5cb
Gerrit-Change-Number: 15028
Gerrit-PatchSet: 1
Gerrit-Owner: Andrew Wong <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 14 Jan 2020 05:21:34 +0000
Gerrit-HasComments: Yes

Reply via email to