Hello Marton Greber, Zoltan Martonka, Alexey Serbin, Kudu Jenkins, Abhishek
Chennaka,
I'd like you to reexamine a change. Please visit
http://gerrit.cloudera.org:8080/24412
to look at the new patch set (#3).
Change subject: KUDU-3429 Add extensive test for delta generation (3/n)
......................................................................
KUDU-3429 Add extensive test for delta generation (3/n)
The patch adds a unit test that is a better version of existing tests
to generate UNDO deltas using:
* Pattern of inserting interleaved rows to ensure more rowsets become
eligible for compaction.
* Wide column data to make each mutation size big enough (64KB). Along
with repetitive updates ensures increased delta memory footprint.
* Increased rows per block batch size that ensures compaction op has
more memory footprint, used to hold rowblock and their mutations.
* Use pseudo-random byte pattern to minimize the effect of compression.
This helps in lowering test runtime possibly because of the time saved
during the delta blocks read that turns into a trivial buffer copy.
* This can act as a de facto unit test for testing high memory compaction
ops. This can be used as a reference base test for any high memory
based compaction tests.
This changelist doesn't contain any functional modifications.
Change-Id: I8c337dba87a9d1cf549f9b53813da8d50e86b6b4
---
M src/kudu/tablet/compaction-highmem-test.cc
M src/kudu/tablet/tablet.h
2 files changed, 272 insertions(+), 0 deletions(-)
git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/12/24412/3
--
To view, visit http://gerrit.cloudera.org:8080/24412
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8c337dba87a9d1cf549f9b53813da8d50e86b6b4
Gerrit-Change-Number: 24412
Gerrit-PatchSet: 3
Gerrit-Owner: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Abhishek Chennaka <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Ashwani Raina <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Marton Greber <[email protected]>
Gerrit-Reviewer: Zoltan Martonka <[email protected]>
