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.lucene.util.InfoStream; +// the testNoStallMergeThreads method will create many files [email protected]("HandleLimitFS") Review Comment: This must never be suppressed for any test. It defeats the entire purposes, which is to detect tests that incorrectly use too many files rather than failing only on some computer systems and not on others. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
