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

Reply via email to