Adar Dembo has posted comments on this change. Change subject: KUDU-921. tablet copy: Make the StartTabletCopy() RPC async ......................................................................
Patch Set 7: > > I'm going to take your advice and do this. I think it will also > > solve the test failure issue. > > OK I am going back and forth on this but I think it's better to > return immediately because we get RPC timeouts otherwise, which > clutter up the logs. I don't have a strong opinion as long as: 1) There's some way to wait for the tablet copy to finish, at least for tests, even if it's super hacky. 2) It's documented in your commit message that the semantics of StartTabletCopy are changing. -- To view, visit http://gerrit.cloudera.org:8080/5045 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-MessageType: comment Gerrit-Change-Id: I95c63f2bfd67624844447862efbdba9cb3676112 Gerrit-PatchSet: 7 Gerrit-Project: kudu Gerrit-Branch: master Gerrit-Owner: Mike Percy <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Dinesh Bhat <[email protected]> Gerrit-Reviewer: Kudu Jenkins Gerrit-Reviewer: Mike Percy <[email protected]> Gerrit-Reviewer: Tidy Bot Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-HasComments: No
