svn commit: r826768 - in /geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents: bin/geronimo.bat bin/geronimo.sh etc/rc.d/start-server, default.groovy

[ke...@apache.org]

Author: kevan
Date: Mon Oct 19 19:10:35 2009
New Revision: 826768

URL: http://svn.apache.org/viewvc?rev=826768&view=rev
Log:
GERONIMO-4885 Update default max heap and max permgen settings. Make gsh and geronimo scripts default to the same settings. Thanks to Quintin Beukes for orignal patch

Modified:
    geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.bat
    geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.sh
    geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/etc/rc.d/start-server,default.groovy

Modified: geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.bat
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.bat?rev=826768&r1=826767&r2=826768&view=diff
==============================================================================
--- geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.bat (original)
+++ geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.bat Mon Oct 19 19:10:35 2009
@@ -177,6 +177,11 @@
 call "%GERONIMO_HOME%\bin\setJavaEnv.bat"
 if not %errorlevel% == 0 goto end
 
+@REM Using default JAVA_OPTS if it's not set
+if not "%JAVA_OPTS%" == "" goto skipDefaultJavaOpts
+set JAVA_OPTS=-Xmx256m -XX:MaxPermSize=128m
+:skipDefaultJavaOpts
+
 if not "%GERONIMO_TMPDIR%" == "" goto gotTmpdir
 @REM A relative value will be resolved relative to each instance 
 set GERONIMO_TMPDIR=var\temp

Modified: geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.sh
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.sh?rev=826768&r1=826767&r2=826768&view=diff
==============================================================================
--- geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.sh (original)
+++ geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/bin/geronimo.sh Mon Oct 19 19:10:35 2009
@@ -201,6 +201,11 @@
   fi
 fi
 
+# Use a default JAVA_OPTS if it's not set
+if [ -z "$JAVA_OPTS" ]; then
+  JAVA_OPTS="-Xmx256m -XX:MaxPermSize=128m"
+fi
+
 if [ -z "$GERONIMO_TMPDIR" ] ; then
   # Define the java.io.tmpdir to use for Geronimo
   # A relative value will be resolved relative to each instance

Modified: geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/etc/rc.d/start-server,default.groovy
URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/etc/rc.d/start-server%2Cdefault.groovy?rev=826768&r1=826767&r2=826768&view=diff
==============================================================================
--- geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/etc/rc.d/start-server,default.groovy (original)
+++ geronimo/server/branches/2.2/framework/configs/geronimo-boilerplate/src/main/underlay/contents/etc/rc.d/start-server,default.groovy Mon Oct 19 19:10:35 2009
@@ -22,15 +22,20 @@
 //
 
 // Append some reasonable java flags if none were configured already
-if (command.profiles.contains('java6')) {
-    command.javaFlags << '-XX:MaxPermSize=512m'
-    command.javaFlags << '-Xmx2048m'
-} else if (command.javaFlags.empty) {
-    command.javaFlags << '-Xmx512m'
+if (command.javaFlags.empty) {
+    command.javaFlags << '-Xmx256m'
+    command.javaFlags << '-XX:MaxPermSize=128m'
 }
 
+// If java version specific configuration were needed, it could take the following
+// form
+// 
+// if (command.profiles.contains('java6')) {
+//    command.javaFlags << '-Xmx2048m'
+// }
+
 // If the debug profile was selected, then append some debugging flags
 if (command.profiles.contains('debug')) {
     command.javaFlags << '-Xdebug'
     command.javaFlags << '-Xrunjdwp:transport=dt_socket,server=y,suspend=n,address=8000'
-}
\ No newline at end of file
+}