Hello Alexey Serbin, Kudu Jenkins,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/21447
to look at the new patch set (#4).
Change subject: Add a benchmark for CBTree concurrent writes.
......................................................................
Add a benchmark for CBTree concurrent writes.
Before updating CBTree for ARM (where it is misbehaving currently),
we should have a proper test for two scenarios:
+ Writing on multiple threads.
+ Reading on multiple threads while there are also active writes.
If read threads wait for values to be inserted, it defeats the purpose
of benchmarking. Therefore, we should first populate a tree with
values for the read threads. The read threads will then read values
that are already in the tree, while the write threads continue to insert
new values.
Setting up the tree for the second scenario essentially involves
performing the first scenario. This is why both scenarios are combined
into a single test.
The new test provides the following new features (compared to just
running DoTestConcurrentInsert with higher parameters):
+ Different threads read the value that inserted it
+ Reader threads can't assigned to a certain writer thread.
+ Keys are better distributed than the previous shuffle method.
+ Allows measuring read-heavy performance (with a flag).
Reading threads start concurrently with writing threads, not at the
end of each write thread (unlike DoTestConcurrentInsert).
Note that running only concurrent reads should not differ from
TestScanPerformance, since no locking takes place and they do not
sabotage each other. So no new test is required for that scenario.
Change-Id: I1b0b16e269c70716962fc5ebb4ddca1e2cbe68a4
---
M src/kudu/tablet/cbtree-test.cc
1 file changed, 184 insertions(+), 1 deletion(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/47/21447/4
--
To view, visit http://gerrit.cloudera.org:8080/21447
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I1b0b16e269c70716962fc5ebb4ddca1e2cbe68a4
Gerrit-Change-Number: 21447
Gerrit-PatchSet: 4
Gerrit-Owner: Zoltan Martonka <zmarto...@cloudera.com>
Gerrit-Reviewer: Alexey Serbin <ale...@apache.org>
Gerrit-Reviewer: Kudu Jenkins (120)
