Merge pull request #2840 from IBM/heap-sizing

Reduce the default min heap size and remove the default max heap size
LinuxForHealth · Oct 15, 2021 · a4a2b29 · a4a2b29
2 parents 837d726 + 5bd4cf5
commit a4a2b29
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 15 deletions.
diff --git a/fhir-server/liberty-config/configDropins/defaults/jvm.options b/fhir-server/liberty-config/configDropins/defaults/jvm.options
@@ -0,0 +1,2 @@
+# Set a fixed, higher than default, initial heap size that could serve a moderate workload
+-Xms768M
diff --git a/fhir-server/liberty-config/configDropins/disabled/jvm.options b/fhir-server/liberty-config/configDropins/disabled/jvm.options
@@ -1,7 +1,8 @@
-# Reduced heap size for constrained environments
+# Set a fixed max heap size
+-Xmx3072M
 
-# Initial heap size (same as max so no reallocations will occur)
--Xms3072M
+# Turn on verbose classloading
+#-verbose:class
 
-# Maximum heap size
--Xmx3072M
+# Turn on verbose garbage collection
+#-verbose:gc
diff --git a/fhir-server/liberty-config/jvm.options b/fhir-server/liberty-config/jvm.options
@@ -14,13 +14,4 @@
 -Dcom.ibm.jsse2.renegotiate=DISABLED
 
 # Prevents Apache Xerces from overactively using the service loader to find a class
--Dorg.apache.xml.dtm.DTMManager=org.apache.xml.dtm.ref.DTMManagerDefault
-
-# Initial heap size (same as max so no reallocations will occur)
--Xms4096M
-
-# Maximum heap size
--Xmx4096M
-
-# Uncomment to turn on verbose classloading
-#-verbose:class
+-Dorg.apache.xml.dtm.DTMManager=org.apache.xml.dtm.ref.DTMManagerDefault