Alexey Serbin has submitted this change and it was merged. ( 
http://gerrit.cloudera.org:8080/18186 )

Change subject: [benchmarks] updated RLE benchmark
......................................................................

[benchmarks] updated RLE benchmark

This patch updates the RLE benchmark to remove some non-RLE operations
(such as IO, memory allocation, etc.) out from the StopWatch scope.

The results of the updated RLE benchmark is used to compare current and
optimized BitUtil::Ceil() implementation: the latter comes in a
follow-up patch.

Change-Id: I8b9df4d20ffb333910f94bbb52a8dcfd0ed6efe9
Reviewed-on: http://gerrit.cloudera.org:8080/18186
Tested-by: Alexey Serbin <[email protected]>
Reviewed-by: Attila Bukor <[email protected]>
---
M src/kudu/benchmarks/rle.cc
1 file changed, 44 insertions(+), 39 deletions(-)

Approvals:
  Alexey Serbin: Verified
  Attila Bukor: Looks good to me, approved

--
To view, visit http://gerrit.cloudera.org:8080/18186
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: merged
Gerrit-Change-Id: I8b9df4d20ffb333910f94bbb52a8dcfd0ed6efe9
Gerrit-Change-Number: 18186
Gerrit-PatchSet: 3
Gerrit-Owner: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Alexey Serbin <[email protected]>
Gerrit-Reviewer: Andrew Wong <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)

Reply via email to