Re: [PR] Fix TestLucene90FieldInfosFormat.testRandom [lucene]

2024-03-11 Thread via GitHub
github-actions[bot] commented on PR #13135: URL: https://github.com/apache/lucene/pull/13135#issuecomment-1989681283 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [PR] Vector accelerated GroupVInt decoder for MemorySegmentIndexInput [lucene]

2024-03-11 Thread via GitHub
github-actions[bot] commented on PR #13133: URL: https://github.com/apache/lucene/pull/13133#issuecomment-1989681309 This PR has not had activity in the past 2 weeks, labeling it as stale. If the PR is waiting for review, notify the d...@lucene.apache.org list. Thank you for your contributi

Re: [I] TestFSTs.testRandomWords reproducible NPE [lucene]

2024-03-11 Thread via GitHub
stefanvodita commented on issue #13174: URL: https://github.com/apache/lucene/issues/13174#issuecomment-1988358935 I ran into this while working on an old branch. I can reproduce it on `main` at commit [ba81826951c1f628efb60c6165add27601173bd9](https://github.com/apache/lucene/commit/ba8182

Re: [I] TestFSTs.testRandomWords reproducible NPE [lucene]

2024-03-11 Thread via GitHub
tteofili commented on issue #13174: URL: https://github.com/apache/lucene/issues/13174#issuecomment-1988348248 I don't seem to be able to reproduce it at the moment, was this observed on `main` branch? -- This is an automated message from the Apache Git Service. To respond to the message,

Re: [I] ArrayIndexOutOfBoundsException in OpenNLPSentenceBreakIterator [lucene]

2024-03-11 Thread via GitHub
tteofili closed issue #12210: ArrayIndexOutOfBoundsException in OpenNLPSentenceBreakIterator URL: https://github.com/apache/lucene/issues/12210 -- 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 s

Re: [PR] Fix middle of sentence calculation in OpenNLPSentenceBreakIterator.preceding [lucene]

2024-03-11 Thread via GitHub
tteofili merged PR #13170: URL: https://github.com/apache/lucene/pull/13170 -- 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.ap

Re: [PR] Fix TestFloatVectorSimilarityQuery.testBoostQuery [lucene]

2024-03-11 Thread via GitHub
benwtrent merged PR #13176: URL: https://github.com/apache/lucene/pull/13176 -- 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.a

Re: [I] Reproducible test failure in TestFloatVectorSimilarityQuery.testBoostQuery [lucene]

2024-03-11 Thread via GitHub
benwtrent closed issue #13173: Reproducible test failure in TestFloatVectorSimilarityQuery.testBoostQuery URL: https://github.com/apache/lucene/issues/13173 -- 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

Re: [PR] Fix NPE when LeafReader return null VectorValues [lucene]

2024-03-11 Thread via GitHub
benwtrent merged PR #13162: URL: https://github.com/apache/lucene/pull/13162 -- 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.a

Re: [PR] An eye-gouging way to limit suppressAccessChecks to just the three JARs that need them. [lucene]

2024-03-11 Thread via GitHub
dweiss commented on PR #13164: URL: https://github.com/apache/lucene/pull/13164#issuecomment-1987964040 I'm for simplifying it, if possible (so removing unnecessary stuff). I don't think security manager coverage is that helpful, especially that security manager is going away in the future

Re: [PR] An eye-gouging way to limit suppressAccessChecks to just the three JARs that need them. [lucene]

2024-03-11 Thread via GitHub
uschindler commented on PR #13164: URL: https://github.com/apache/lucene/pull/13164#issuecomment-1987929221 P.S.: If we remove support for SecurityManager in combination with EMMA, we should remove the already existing policy file entries that are related to EMMA. I think there are a few. M

Re: [PR] An eye-gouging way to limit suppressAccessChecks to just the three JARs that need them. [lucene]

2024-03-11 Thread via GitHub
uschindler commented on PR #13164: URL: https://github.com/apache/lucene/pull/13164#issuecomment-1987926278 I will check a bit later, sorry time is limited. -- 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

Re: [PR] An eye-gouging way to limit suppressAccessChecks to just the three JARs that need them. [lucene]

2024-03-11 Thread via GitHub
dweiss commented on PR #13164: URL: https://github.com/apache/lucene/pull/13164#issuecomment-1987848703 Probably possible - not sure whether increasing the build logic is worth it. Maybe it (coverage task) should just fail if it detects conflicting options? -- This is an automated message