Skip to content
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

Set also MaxMetaspace on Java 8 #12

Open
rhuss opened this issue Dec 9, 2016 · 4 comments
Open

Set also MaxMetaspace on Java 8 #12

rhuss opened this issue Dec 9, 2016 · 4 comments

Comments

@rhuss
Copy link
Contributor

rhuss commented Dec 9, 2016

If running under Java 8, the Metaspace should be limited, too in --> https://github.com/fabric8io-images/run-java-sh/blob/master/fish-pepper/run-java-sh/fp-files/java-default-options#L46

@rhuss
Copy link
Contributor Author

rhuss commented Nov 12, 2017

@astefanutti wdyt about limiting the MaxMetaspace, too ?

@astefanutti
Copy link
Contributor

Yes, that makes sense. Let me try working something out.

As we go from sizing from one region (the heap), to multiple regions (metaspace, and JIT code maybe), it becomes tricker to heuristically size them generally. At least each one can be bound to max memory minus heap size.

@derkoe
Copy link
Contributor

derkoe commented Mar 6, 2018

Don't forget direct memory used by NIO buffers: -XX:MaxDirectMemorySize. See https://docs.oracle.com/cd/E15289_01/doc.40/e15062/optionxx.htm#BABGCFFB

@rhuss
Copy link
Contributor Author

rhuss commented Mar 17, 2018

Thanks ! wonder how this could be combined with the -xmx heap settings so that the combined value (+ all the other non-heap memory) doesn't exceed the given constraints.

I think we can't give a good default value and as this limit is unbounded by default anyway, I think also that its fair to require that the users have to specify this limit on their own when they require it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants