typo in jmxtrans.sh

[GoogleCodeExporter]

I just see a small typo on jmxtrans.sh 

NEW_SIZE=${NEW_SIZE:-"64M"}
NEW_RATIO=${NEW_RATIO:-"8"}
HEAP_SIZE=${HEAP_SIZE:-"512M"}
CPU_CORES=${CPU_CORES:-"1"}
MONITOR_OPTS=${MONITOR_OPTS:-"-Dcom.sun.management.jmxremote 
-Dcom.sun.management.jmxremote.ssl=false 
-Dcom.sun.management.jmxremote.authenticate=false 
-Dcom.sun.management.jmxremote.port=2101"}
GC_OPTS=${GC_OPTS:-"-Xms${HEAP_SIZE}M -Xmx${HEAP_SIZE}M -XX:+UseConcMarkSweepGC 
-XX:NewRatio=${NEW_RATIO} -XX:NewSize=${NEW_SIZE}m -XX:MaxNewSize=${NEW_SIZE}m 
-XX:MaxTenuringThreshold=16 -XX:GCTimeRatio=9 -XX:PermSize=384m 
-XX:MaxPermSize=384m -XX:+UseTLAB -XX:CMSInitiatingOccupancyFraction=85 
-XX:+CMSIncrementalMode -XX:+CMSIncrementalPacing 
-XX:ParallelGCThreads=$CPU_CORES -Dsun.rmi.dgc.server.gcInterval=28800000 
-Dsun.rmi.dgc.client.gcInterval=28800000"}

NEW_SIZE and HEAP_SIZE allready include the M (for MB)
It should be removed from -Xms${HEAP_SIZE}M -Xmx${HEAP_SIZE}M

Original issue reported on code.google.com by henri.gomez on 1 Jul 2011 at 9:54

[GoogleCodeExporter]

And also from :

-XX:NewSize=${NEW_SIZE}M and  -XX:MaxNewSize=${NEW_SIZE}M

Original comment by henri.gomez on 1 Jul 2011 at 9:55

[GoogleCodeExporter]

Fixed in r164

Original comment by latch...@gmail.com on 1 Jul 2011 at 3:55

• Changed state: Fixed