Skip to content

scaladoc OOMs while building ScalaTest 3.2.7 docs in community build #12124

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
griggt opened this issue Apr 16, 2021 · 8 comments
Closed

scaladoc OOMs while building ScalaTest 3.2.7 docs in community build #12124

griggt opened this issue Apr 16, 2021 · 8 comments

Comments

@griggt
Copy link
Contributor

griggt commented Apr 16, 2021

https://github.com/lampepfl/dotty/runs/2362395012?check_suite_focus=true#step:5:2235

java.lang.RuntimeException: Doc command exited with code 137 for project scalatest. Project details:
SbtCommunityProject(scalatest,scalacticDotty/clean; scalacticDottyJS/clean; scalacticTestDotty/test; scalatestTestDotty/test; scalacticDottyJS/compile; scalatestDottyJS/compile,List(),List(SbtCommunityProject(scala-xml,xml/test,List(),List(),dotty.communitybuild.SbtCommunityProject$$$Lambda$23/1725097945@7cd62f43,xml/publishLocal,xml/doc,List(-Xcheck-macros, -Ysafe-init),false)),dotty.communitybuild.projects$$$Lambda$27/1458849419@6d4b1c02,scalacticDotty/publishLocal; scalatestDotty/publishLocal; scalacticDottyJS/publishLocal; scalatestDottyJS/publishLocal,;scalacticDotty/doc,List(-Xcheck-macros, -Ysafe-init),false)
	at dotty.communitybuild.CommunityProject.doc(projects.scala:79)
	at dotty.communitybuild.CommunityProject.doc$(projects.scala:35)
	at dotty.communitybuild.SbtCommunityProject.doc(projects.scala:104)
	at dotty.communitybuild.Main$.generateDocs(Main.scala:14)
	at dotty.communitybuild.Main$.$anonfun$6(Main.scala:67)
	at scala.collection.immutable.List.map(List.scala:250)
	at dotty.communitybuild.Main$.main(Main.scala:78)
	at dotty.communitybuild.Main.main(Main.scala)

Sometimes it completes without error.

It works fine for me locally, but I have quite a bit more RAM than the 7GB (?) that I think the GitHub hosted runners provide.

@smarter
Copy link
Member

smarter commented Apr 16, 2021

Oh, so exit 137 means that docker was OOM, not the JVM (where the max heap for scalatest is set to 11 GB) ? That's going to be hard to work around.

@griggt
Copy link
Contributor Author

griggt commented Apr 16, 2021

Yep

@abgruszecki
Copy link
Contributor

OK, this was fixed in #12145. Guillaume said that compiling ScalaTest also takes an excessive amount of memory, so I'm not sure if it makes sense to investigate the problems that Scaladoc has with ScalaTest before the compiler's own problems are fixed.

@abgruszecki
Copy link
Contributor

@smarter do we have an issue for the OOM when compiling ScalaTest? If yes, then maybe we should keep this one open and link it there.

@smarter
Copy link
Member

smarter commented Apr 19, 2021

I don't think so

@griggt
Copy link
Contributor Author

griggt commented Apr 19, 2021

IIRC it's compiling the ScalaTest test suite that uses excessive memory, rather than compiling ScalaTest proper.

@abgruszecki
Copy link
Contributor

Ok, let's at least re-open this issue and check whether Scaladoc's problems are the same as compiler's.

@abgruszecki abgruszecki reopened this Apr 20, 2021
@griggt
Copy link
Contributor Author

griggt commented Jul 11, 2021

This no longer seems to be an issue if we reduce the JVM max heap size for ScalaTest from 11 GB to 5 GB (which is now possible thanks to #12935).

See #13049.

@griggt griggt closed this as completed Jul 11, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants