maedhroz commented on code in PR #2935:
URL: https://github.com/apache/cassandra/pull/2935#discussion_r1496385178
##########
test/unit/org/apache/cassandra/index/sai/cql/intersection/RandomIntersectionTester.java:
##########
@@ -80,7 +80,8 @@ protected void runRestrictedQueries() throws Throwable
Map<Integer, List<TestRow>> testRowMap = buildAndLoadTestRows();
beforeAndAfterFlush(() -> {
- for (int queryCount = 0; queryCount < nextInt(10, 100);
queryCount++)
+ int queryCount = nextInt(10, 80);
Review Comment:
So I moved this out of the loop condition because it was re-evaluating on
every iteration rather than being set before the loop, as I'm sure we
originally intended. I think `10-80` ended up producing a distribution pretty
similar to what was there before. (i.e. We very rarely ever got to the upper
end of the distribution with re-evaluating on every iteration.)
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
