Bankim Bhavsar has posted comments on this change. ( http://gerrit.cloudera.org:8080/16064 )
Change subject: [util] add BlockingQueueTest.MultiThreadPerf test scenario ...................................................................... Patch Set 2: Code-Review+2 (1 comment) http://gerrit.cloudera.org:8080/#/c/16064/1/src/kudu/util/blocking_queue-test.cc File src/kudu/util/blocking_queue-test.cc: http://gerrit.cloudera.org:8080/#/c/16064/1/src/kudu/util/blocking_queue-test.cc@45 PS1, Line 45: DEFINE_uint32(num_blocking_writers, 3, : "number of threads calling BlockingQueue::BlockingPut()"); : DEFINE_uint32(num_non_blocking_writers, 2, : "number of threads calling BlockingQueue::Put()"); : DEFINE_uint32(num_blocking_readers, 5, : "number of threads calling BlockingQueue::BlockingGet()"); : DEFINE_uint32(runtime_sec, 5, "duration of the test (seconds)"); : DEFINE_uint32(queue_capacity, 64, "capacity of the queue (number of elements)"); > The reason I added these flags is to be able include this test into src/kud Thanks for the explanation. -- To view, visit http://gerrit.cloudera.org:8080/16064 To unsubscribe, visit http://gerrit.cloudera.org:8080/settings Gerrit-Project: kudu Gerrit-Branch: master Gerrit-MessageType: comment Gerrit-Change-Id: If52e4e15d3c72ad3479334f0f3fa8bb10b8c50c6 Gerrit-Change-Number: 16064 Gerrit-PatchSet: 2 Gerrit-Owner: Alexey Serbin <[email protected]> Gerrit-Reviewer: Alexey Serbin <[email protected]> Gerrit-Reviewer: Andrew Wong <[email protected]> Gerrit-Reviewer: Bankim Bhavsar <[email protected]> Gerrit-Reviewer: Kudu Jenkins (120) Gerrit-Reviewer: Tidy Bot (241) Gerrit-Reviewer: Todd Lipcon <[email protected]> Gerrit-Comment-Date: Thu, 11 Jun 2020 20:11:37 +0000 Gerrit-HasComments: Yes
