Mike Percy has posted comments on this change.

Change subject: tablet copy: Make the StartTabletCopy() RPC async
......................................................................


Patch Set 2:

This needs a couple of changes, I think:

1. Increase the minimum number of threads on the open_tablet_pool_ or possibly 
use a different pool. I was thinking 4 threads minimum, maybe 4 + # disks

2. I think we need to run the whole process on the thread pool because if there 
is a long delay between opening the tablet bootstrap session and finishing it, 
the session could time out on the remote and the process would fail.

-- 
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: 2
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Mike Percy <[email protected]>
Gerrit-Reviewer: Dinesh Bhat <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-HasComments: No

Reply via email to