gharris1727 commented on PR #12806: URL: https://github.com/apache/kafka/pull/12806#issuecomment-1299120891
@divijvaidya Thanks for pointing me to that PR with a similar flakey test. I tried applying this bound to the situation in that test and it didn't hold, and managed to replicate the behavior in my unit test with a parameterized test. Here's the results, just for discussion's sake: I was testing the minInterval=0 case when I was developing this bound, but it appears that the bound is incorrect in more general situations. ``` [1] numSamples=1, minInterval=0 [2] numSamples=1, minInterval=1 [3] numSamples=1, minInterval=2 [4] numSamples=1, minInterval=10 [5] numSamples=1, minInterval=20 [6] numSamples=1, minInterval=30 [7] numSamples=1, minInterval=39 [8] numSamples=1, minInterval=40 [9] numSamples=2, minInterval=0 PASS [10] numSamples=2, minInterval=1 [11] numSamples=2, minInterval=2 [12] numSamples=2, minInterval=10 [13] numSamples=2, minInterval=20 [14] numSamples=2, minInterval=30 [15] numSamples=2, minInterval=39 [16] numSamples=2, minInterval=40 PASS [17] numSamples=10, minInterval=0 PASS [18] numSamples=10, minInterval=1 PASS [19] numSamples=10, minInterval=2 [20] numSamples=10, minInterval=10 [21] numSamples=10, minInterval=20 [22] numSamples=10, minInterval=30 [23] numSamples=10, minInterval=39 [24] numSamples=10, minInterval=40 PASS ``` -- 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: jira-unsubscr...@kafka.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
