Will Berkeley has posted comments on this change. ( http://gerrit.cloudera.org:8080/12613 )
Change subject: KUDU-2711 (part 1): add a benchmark for GetTableLocations ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/12613/1/src/kudu/integration-tests/table_locations-itest.cc File src/kudu/integration-tests/table_locations-itest.cc: http://gerrit.cloudera.org:8080/#/c/12613/1/src/kudu/integration-tests/table_locations-itest.cc@326 PS1, Line 326: const int kNumReactors = 8; > I think I went with multiple whole messengers so that we end up with multip Gotcha, we should probably keep it straightforward and match the number of messengers to the number of threads, then. -- To view, visit http://gerrit.cloudera.org:8080/12613 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic1643638d82b7697d20ea960592c8b70c30ac7bd Gerrit-Change-Number: 12613 Gerrit-PatchSet: 1 Gerrit-Owner: Todd Lipcon <[email protected]> Gerrit-Reviewer: Adar Dembo <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Reviewer: Will Berkeley <[email protected]> Gerrit-Comment-Date: Thu, 07 Mar 2019 18:14:14 +0000 Gerrit-HasComments: Yes
