You are viewing a plain text version of this content. The canonical link for it is here.
Posted to reviews@kudu.apache.org by "Yingchun Lai (Code Review)" <ge...@cloudera.org> on 2022/05/05 02:42:02 UTC

[kudu-CR] [tools] use thread pool for 'kudu local replica copy from local'

Hello Alexey Serbin, Yifan Zhang, Kudu Jenkins, Andrew Wong, 

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/18448

to look at the new patch set (#6).

Change subject: [tools] use thread pool for 'kudu local_replica copy_from_local'
......................................................................

[tools] use thread pool for 'kudu local_replica copy_from_local'

In real world, we often use copy_from_local to copy a large
amount of tablets, we can pass a tablet id list to the tool
for convenient, and we can also gain profit of opening
filesystem only once to reduce time.

This patch also updates the related unit test.

Change-Id: I4de49e948cc1a686db5e1bf424470ca9e800ee36
---
M src/kudu/tools/kudu-tool-test.cc
M src/kudu/tools/tool_action_common.cc
M src/kudu/tools/tool_action_local_replica.cc
3 files changed, 122 insertions(+), 42 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/48/18448/6
-- 
To view, visit http://gerrit.cloudera.org:8080/18448
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I4de49e948cc1a686db5e1bf424470ca9e800ee36
Gerrit-Change-Number: 18448
Gerrit-PatchSet: 6
Gerrit-Owner: Yingchun Lai <ac...@gmail.com>
Gerrit-Reviewer: Alexey Serbin <al...@apache.org>
Gerrit-Reviewer: Andrew Wong <aw...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Yifan Zhang <ch...@163.com>
Gerrit-Reviewer: Yingchun Lai <ac...@gmail.com>