Abhishek Chennaka has posted comments on this change. ( http://gerrit.cloudera.org:8080/21212 )
Change subject: [rpc] relax settings for MeasureAcceptorDispatchTimes ...................................................................... Patch Set 1: (1 comment) http://gerrit.cloudera.org:8080/#/c/21212/1/src/kudu/rpc/rpc-bench.cc File src/kudu/rpc/rpc-bench.cc: http://gerrit.cloudera.org:8080/#/c/21212/1/src/kudu/rpc/rpc-bench.cc@354 PS1, Line 354: for (auto i = 0; i < threads_num; ++i) { Do we want to get rid of this loop and modify other areas of this test since we only have one thread now or leave it as is in case we want to increase the number of threads in future for any reason? -- To view, visit http://gerrit.cloudera.org:8080/21212 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: Ic9af59046d6bcf28134cc88e7ff9e54643efc37f Gerrit-Change-Number: 21212 Gerrit-PatchSet: 1 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Abhishek Chennaka <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Comment-Date: Thu, 28 Mar 2024 06:19:05 +0000 Gerrit-HasComments: Yes
