Hello Mahesh Reddy, Tidy Bot, Zoltan Martonka, Kudu Jenkins, Wang Xixu,

I'd like you to reexamine a change. Please visit

    http://gerrit.cloudera.org:8080/20899

to look at the new patch set (#8).

Change subject: [log_block_manager-test] Improve random selection
......................................................................

[log_block_manager-test] Improve random selection

The previous solution to remove random blocks was not good, because it
was possible that way less blocks are removed than expected because it
is possible that rand()%100 returns 0 or 99 every time and that means in
extreme cases zero or all the elements are deleted. The new approach
removes exactly the same number of blocks every time but randomizes
which blocks are removed and the order of removal is randomized, too.

Change-Id: I9f73f0572319784b07845b02d15a7d3e6f31abce
---
M src/kudu/fs/log_block_manager-test.cc
1 file changed, 5 insertions(+), 5 deletions(-)


  git pull ssh://gerrit.cloudera.org:29418/kudu refs/changes/99/20899/8
--
To view, visit http://gerrit.cloudera.org:8080/20899
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I9f73f0572319784b07845b02d15a7d3e6f31abce
Gerrit-Change-Number: 20899
Gerrit-PatchSet: 8
Gerrit-Owner: Ádám Bakai <aba...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Reviewer: Mahesh Reddy <mre...@cloudera.com>
Gerrit-Reviewer: Tidy Bot (241)
Gerrit-Reviewer: Wang Xixu <1450306...@qq.com>
Gerrit-Reviewer: Zoltan Martonka <zmarto...@cloudera.com>
Gerrit-Reviewer: Ádám Bakai <aba...@cloudera.com>

Reply via email to