cloudfoundry / java-buildpack-memory-calculator

Cloud Foundry JVM Memory Calculator
Apache License 2.0
645 stars 70 forks source link

GC parallelism tweaks #14

Closed jtwaleson closed 6 years ago

jtwaleson commented 6 years ago

I recently read this article which points out that the garbage collector might need to be tuned (XX:ParallelGCThreads XX:ConcGCThreads) to the number of CPUs in containers. Any thoughts about including this in the memory calculator?

nebhale commented 6 years ago

We've not seen this being a huge problem at the moment, so I'm not inclined to make a change here. If you're having trouble cf set-env <APP> JAVA_OPTS '...' will give you the behavior you want, but until we start seeing people getting enough unanticipated threads that they get OOMs, I think we hold fire.

glyn commented 6 years ago

Closing for now. Please re-open if necessary.