Re: [PR] Initial impl of MMapDirectory for Java 22 [lucene]

2024-01-13 Thread via GitHub
uschindler commented on PR #12706: URL: https://github.com/apache/lucene/pull/12706#issuecomment-1890455325 I also committed supprot for incubator SIMD vectorization in Java 22. According to Java's change logs there were no changes to API at all, so code runs as is. @ChrisHegarty jus

Re: [PR] Initial impl of MMapDirectory for Java 22 [lucene]

2024-01-04 Thread via GitHub
uschindler commented on PR #12706: URL: https://github.com/apache/lucene/pull/12706#issuecomment-1877096268 No no no, I am not stale! -- 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

Re: [PR] Initial impl of MMapDirectory for Java 22 [lucene]

2023-12-01 Thread via GitHub
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/apac

Re: [PR] Initial impl of MMapDirectory for Java 22 [lucene]

2023-11-30 Thread via GitHub
uschindler commented on PR #12706: URL: https://github.com/apache/lucene/pull/12706#issuecomment-1833435917 After https://github.com/jdk/pull/16792 was fixed, I added the better isAlive check to the Java 21+ code. -- This is an automated message from the Apache Git Service. To respond to

Re: [PR] Initial impl of MMapDirectory for Java 22 [lucene]

2023-10-22 Thread via GitHub
uschindler commented on PR #12706: URL: https://github.com/apache/lucene/pull/12706#issuecomment-1774030650 I updated the Jenkins jobs running mmap tests to use this branch: https://jenkins.thetaphi.de/job/Lucene-MMAPv2-Linux/, https://jenkins.thetaphi.de/job/Lucene-MMAPv2-Windows/ -- Th