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 +@LuceneTestCase.SuppressFileSystems("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: issues-unsubscr...@lucene.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org --------------------------------------------------------------------- To unsubscribe, e-mail: issues-unsubscr...@lucene.apache.org For additional commands, e-mail: issues-h...@lucene.apache.org