[
https://issues.apache.org/jira/browse/SLING-6756?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15949073#comment-15949073
]
Robert Munteanu commented on SLING-6756:
----------------------------------------
Do you have an example which shows that setting the max heap size will lead to
problems? Initially 512m was recommended for the whole reactor build.
> Remove setting/advising to set permsize and max heap size
> ---------------------------------------------------------
>
> Key: SLING-6756
> URL: https://issues.apache.org/jira/browse/SLING-6756
> Project: Sling
> Issue Type: Improvement
> Reporter: Konrad Windszus
>
> PermSize is no longer an issue with java 8 as Java 8 no longer knows a
> dedicated permgen space
> (https://dzone.com/articles/java-8-permgen-metaspace). Still we set
> {{-XX:MaxPermSize=256m}} in
> https://github.com/apache/sling/blob/trunk/.mvn/jvm.config (SLING-4530) which
> leads to a warning when building with Java 8.
> The related ant check in parent has been removed though in
> https://issues.apache.org/jira/browse/SLING-5862. Althought the reason why
> the check has been removed is a bit unclear to me from the comments, I would
> propose to remove the jvm.config completely and do no longer advise to set
> permsize or heapsize explicitly.
> The heapsize is currently bound to at most 512MB, which is in 90% of the
> cases just limiting the maximum heapsize of a modern JVM (because it should
> be 1/4 of the physical memory in most of the JVM, see
> http://stackoverflow.com/questions/4667483/how-is-the-default-java-heap-size-determined).
--
This message was sent by Atlassian JIRA
(v6.3.15#6346)
