Alexey Serbin has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/10542 )

Change subject: [tools] KUDU-2413 Configurable # of reactors in loadgen
......................................................................


Patch Set 2:

(2 comments)

http://gerrit.cloudera.org:8080/#/c/10542/2//COMMIT_MSG
Commit Message:

http://gerrit.cloudera.org:8080/#/c/10542/2//COMMIT_MSG@9
PS2, Line 9: loadgen which allows
           : configuring the number of reactors on the Messenger
How much benefits does it bring?  I.e., what are the scenarios which require 
custom number of reactor threads for the RPC messenger?  It would be nice to 
see some comparison for the default and the custom number.


http://gerrit.cloudera.org:8080/#/c/10542/2/src/kudu/tools/tool_action_perf.cc
File src/kudu/tools/tool_action_perf.cc:

http://gerrit.cloudera.org:8080/#/c/10542/2/src/kudu/tools/tool_action_perf.cc@202
PS2, Line 202: to be used
for the RPC messenger of the KuduClient



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

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: I8f02e6ac153ff58c2d3a89a9966ea6fe721db06e
Gerrit-Change-Number: 10542
Gerrit-PatchSet: 2
Gerrit-Owner: Attila Bukor <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Will Berkeley <[email protected]>
Gerrit-Comment-Date: Wed, 30 May 2018 16:23:48 +0000
Gerrit-HasComments: Yes

Reply via email to