Skip to content

Commit 46588e7

Browse files
smarterDarkDimius
authored andcommitted
Stop crashes because we're out of memory by disabling t7880
All of our recent memory-related tests failures since scala#1030 was merged seem to be caused by t7880.scala. It tries to intentionally trigger an OutOfMemoryError, however since we don't pass -Xmx to our run tests it's possible that this we fill up the memory of our host before we reach the maximum heap size of the JVM. Ideally, we would specify a -Xmx for run tests (scalac uses 1 GB), unfortunately in the version of partest we use this is tricky because we need to set the system property "partest.java_opts". If we upgrade our partest to the latest release, we can instead specify it by setting the argument `javaOpts` of the constructor of `SuiteRunner`, see scala/scala-partest@7c4659e
1 parent adfdbd6 commit 46588e7

File tree

1 file changed

+1
-0
lines changed

1 file changed

+1
-0
lines changed

tests/run/t7880.scala renamed to tests/pending/run/t7880.scala

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
1+
// Do "git log" on this file to know why it's been moved to pending
12
object Test extends dotty.runtime.LegacyApp {
23
// This should terminate in one way or another, but it shouldn't loop forever.
34
try {

0 commit comments

Comments
 (0)