epugh commented on PR #5029: URL: https://github.com/apache/solr/pull/5029#issuecomment-6023504350
> 🤖 _AI text below_ 🤖 _(posted on behalf of Nick Shanin)_ > > A bit of context on why this PR exists: the failure first surfaced on my own PR #5013's Crave run, where testSimple failed in a tree that does not touch this code. The root cause check cleared that PR, and a sweep of recent CI failures then showed 14 occurrences in 30 days across unrelated PRs, ten of them on #4976. That is what turned it into a ticket and this fix. > > Nothing changes in this PR from #4976 merging: it targets main, which still pins Caffeine 3.2.4, and #4976 landed only on branch_9x with version pins and no source changes, so the fix and its evidence here are untouched. The one effect is on the backport side mentioned in Limits: branch_9x now runs Caffeine 3.3.0, so if a branch_9x backport is wanted, its accounting read would be done against 3.3.0 rather than 3.2.4. The fix shape itself (capacity above anything the test can hold during warming) is the same on either version, and I am still happy to prepare that backport on request. I'm going to channel my inner David Smiley in this comment: I don't think I needed two paragraphs of AI generated text to answer this question. a "Nope, this test is still valid on main and branch_10x,not on 9x" would have been enough ;-). The wall of words is tough to digest at times. -- 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]
