uschindler commented on PR #12706: URL: https://github.com/apache/lucene/pull/12706#issuecomment-1836540146
I tested the new code - and at the same time commenting out the Java 21 fallback code to check for "closed" in the `IllegalStateException` message here: https://github.com/apache/lucene/blob/5c9152917614ecee0eef7612ab71c24c9d92529c/lucene/core/src/java21/org/apache/lucene/store/MemorySegmentIndexInput.java#L116-L121 With JDK 22 build 23 the test fails because it rethrows the `IllegalStateException` due to the `isAlive()` check didn't work. With build 26 beasting the test worked for hours: ``` $ ./gradlew :lucene:core:beast --tests TestMMapDirectory.testAceWithThreads -Ptests.multiplier=500 -Ptests.dups=100 ``` (build 23 or earlier Java versions like 21 failed on first or second build) -- 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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org
