Todd Lipcon has submitted this change and it was merged.

Change subject: rpc-bench: add async benchmark, make more configurable
......................................................................


rpc-bench: add async benchmark, make more configurable

This changes rpc-bench to have an 'async' mode which uses a smaller
number of client threads which have multiple outstanding calls.
It also now reports the number of context switches per call as well
as the parameters used to run the test.

Baseline results on a 24 core 2.5Ghz (48 logical cores) system:

./bin/rpc-bench -server_reactors 24 -async_call_concurrency 100 -worker_threads 
40 -client_threads 24  -run_seconds 3

  Mode:            Sync
  Client threads:   24
  Worker threads:   40
  Server reactors:  24
  ----------------------------------
  Reqs/sec:         189006
  User CPU per req: 21.9328us
  Sys CPU per req:  31.633us
  Ctx Sw. per req:  5.08355

  Mode:            Async
  Client reactors:  24
  Call concurrency: 100
  Worker threads:   40
  Server reactors:  24
  ----------------------------------
  Reqs/sec:         202874
  User CPU per req: 14.2749us
  Sys CPU per req:  23.538us
  Ctx Sw. per req:  2.7318

Change-Id: I8318911ea76d85fa96acb52203101a97a539e42c
Reviewed-on: http://gerrit.cloudera.org:8080/2937
Tested-by: Kudu Jenkins
Reviewed-by: Adar Dembo <[email protected]>
---
M src/kudu/rpc/rpc-bench.cc
M src/kudu/scripts/benchmarks.sh
M src/kudu/util/stopwatch.h
3 files changed, 151 insertions(+), 21 deletions(-)

Approvals:
  Adar Dembo: Looks good to me, approved
  Kudu Jenkins: Verified



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I8318911ea76d85fa96acb52203101a97a539e42c
Gerrit-PatchSet: 4
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <[email protected]>
Gerrit-Reviewer: Adar Dembo <[email protected]>
Gerrit-Reviewer: Binglin Chang <[email protected]>
Gerrit-Reviewer: Henry Robinson <[email protected]>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Mike Percy <[email protected]>
Gerrit-Reviewer: Todd Lipcon <[email protected]>

Reply via email to