Re: [PR] Do not use HandleLimitFS for TestConcurrentMergeScheduler [lucene]

2024-01-26 Thread via GitHub
easyice commented on PR #13035: URL: https://github.com/apache/lucene/pull/13035#issuecomment-1912109709 Thank you! i replaced the suppress with `setMaxBufferedDocs`, the open files in nightly tests reduced from 4000+ to 400+, does that looks okay? The `testDeleteMerging` might also c

Re: [PR] Do not use HandleLimitFS for TestConcurrentMergeScheduler [lucene]

2024-01-26 Thread via GitHub
easyice commented on code in PR #13035: URL: https://github.com/apache/lucene/pull/13035#discussion_r1467674266 ## lucene/core/src/test/org/apache/lucene/index/TestConcurrentMergeScheduler.java: ## @@ -43,6 +43,8 @@ import org.apache.lucene.tests.util.TestUtil; import org.apac

Re: [PR] Do not use HandleLimitFS for TestConcurrentMergeScheduler [lucene]

2024-01-26 Thread via GitHub
rmuir commented on code in PR #13035: URL: https://github.com/apache/lucene/pull/13035#discussion_r1467598866 ## lucene/core/src/test/org/apache/lucene/index/TestConcurrentMergeScheduler.java: ## @@ -43,6 +43,8 @@ import org.apache.lucene.tests.util.TestUtil; import org.apache

Re: [PR] Do not use HandleLimitFS for TestConcurrentMergeScheduler [lucene]

2024-01-26 Thread via GitHub
rmuir commented on PR #13035: URL: https://github.com/apache/lucene/pull/13035#issuecomment-1911992774 This is not the correct fix: instead the test must be fixed to not use so many files at once. -- This is an automated message from the Apache Git Service. To respond to the message, plea

[PR] Do not use HandleLimitFS for TestConcurrentMergeScheduler [lucene]

2024-01-26 Thread via GitHub
easyice opened a new pull request, #13035: URL: https://github.com/apache/lucene/pull/13035 ``` ./gradlew :lucene:core:test --tests "org.apache.lucene.index.TestConcurrentMergeScheduler.testNoStallMergeThreads" -Ptests.jvms=6 "-Ptests.jvmargs=-XX:TieredStopAtLevel=1 -XX:+UseParallelGC -