Todd Lipcon has posted comments on this change.

Change subject: KUDU-1590. Fix cache-test failure on some machines
......................................................................


Patch Set 1:

(1 comment)

http://gerrit.cloudera.org:8080/#/c/4321/1/src/kudu/util/cache-test.cc
File src/kudu/util/cache-test.cc:

Line 190:   for (int i = 0; i < kNumElems + 1000; i++) {
> Just for my own curiosity (don't need to change anything), what exactly is 
Didn't really dig into this... my guess is that it was right "on the edge" 
here, and the number of shards just shuffled the elements we're touching here 
around differently such that sometimes '101' didn't get evicted, and sometimes 
it did.


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

Gerrit-MessageType: comment
Gerrit-Change-Id: I77362fc67ab5ba8420c21d1e3fb5c28ff9bfab1b
Gerrit-PatchSet: 1
Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-Owner: Todd Lipcon <t...@apache.org>
Gerrit-Reviewer: Adar Dembo <a...@cloudera.com>
Gerrit-Reviewer: Kudu Jenkins
Gerrit-Reviewer: Todd Lipcon <t...@apache.org>
Gerrit-HasComments: Yes

Reply via email to