Re: [PR] Fix too many open files Exception for TestConcurrentMergeScheduler [lucene]

2024-01-30 Thread via GitHub
easyice commented on PR #13035: URL: https://github.com/apache/lucene/pull/13035#issuecomment-1917236900 Pushed a new fix for reproducible test failure `TestIndexWriterThreadsToSegments.testManyThreadsClose`: ``` ./gradlew test --tests TestIndexWriterThreadsToSegments.testManyT

Re: [PR] Fix too many open files Exception for TestConcurrentMergeScheduler [lucene]

2024-01-27 Thread via GitHub
easyice commented on PR #13035: URL: https://github.com/apache/lucene/pull/13035#issuecomment-1913090991 It looks better to use `TestUtil.reduceOpenFiles` for `testDeleteMerging`. In `testNoStallMergeThreads`, it need many segment to cover LUCENE-6197(i tried to checkout the old version

Re: [PR] Fix too many open files Exception for TestConcurrentMergeScheduler [lucene]

2024-01-26 Thread via GitHub
rmuir commented on PR #13035: URL: https://github.com/apache/lucene/pull/13035#issuecomment-1912510553 Thank you @easyice ! There is also a `TestUtil.reduceOpenFiles`, I'm not sure if it is appropriate here, but something to look into. It is also nice since it makes it obvious from the code