mikemccand commented on PR #15585: URL: https://github.com/apache/lucene/pull/15585#issuecomment-3820113225
Thanks @uschindler and @dweiss. Actually, now I'm wondering about thread contention ... `AtomicInteger` isn't magic, it still must obey the physics / mechanical sympathy down at the CPU core level. OK I asked Claude Opus 4.5 about this: https://claude.ai/share/a9ddcc42-871c-4959-beac-c7b1a90bae0b Net/net it seems like we don't need to worry about that. Yes, contention is bad ("MESI ping-pong!!"), and even more-so on ARM than recent x86-64 CPUs, but the amount of contention we're adding here should be tiny, and the crazy cost it is fixing (over-zealous `isLoaded()`) was much worse. So we're good! I'm trying to get nightlies working again so we can see if needles moved... -- 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]
