search

cloudfoundry / java-buildpack-memory-calculator

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

Documented formula for estimating metaspace appears to be incorrect #23

Closed vpavic closed 5 years ago

vpavic commented 5 years ago

The document Java Buildpack Memory Calculator v3 linked in project's readme contains the following formula for estimating metaspace:

5400 * (number of loaded casses) + 7000000

This appears to be out of sync with:

https://github.com/cloudfoundry/java-buildpack-memory-calculator/blob/beef546da73761e7d1e1cfaacac6ff8655ad40aa/calculator/calculator.go#L103-L105

nebhale commented 5 years ago

Yeah, that's what it was with permgen based versions although even that hasn't kept up with our evolution.