thanks for the initiative. you will definitely get svn access in case you plan to contribute fixes and enhancements to OJ core. bear in mind that you should announce API changes and discuss them before implementing any. also generally we regard backward compatibility quite highly, so generally we leave old API in and only tag it deprecated most of the time.
wrt. your patch. i am not really satisfied with the shell coding style, but the approach seems sound to me. major issue to me is that 80% of e.g. 16Gb leaves 3.2GB unused. as XMX only defines a soft limit i'd tend to make all RAM available to java, like i did it for windows. probably most elegant would be to postprocess the value, similar to the 64bit case and simply decide to either use 80% or RAM-1GB, whichever is bigger. give me some time to integrate your patch into the linux start script. maybe i'll come up with a solution for mac as well while being at it. ..ede On 09.02.2014 20:11, Larry Reeder wrote: > Hey Ede, > > I updated oj_linux.sh so that it uses up to 80% of available RAM on Linux > instead of defaulting to 512M. > > Tested with 32 and 64 bit JVM (only allows max 3GB on 32 bit JVM). > Tested on Mac OSX - doesn't do anything, but doesn't break the script either. > > Patch is attached. I don't have write access to the OJ repo. Can you apply > the patch or give me write access and I'll apply it myself? > > > Thanks...... lreeder > > > ------------------------------------------------------------------------------ > Managing the Performance of Cloud-Based Applications > Take advantage of what the Cloud has to offer - Avoid Common Pitfalls. > Read the Whitepaper. > http://pubads.g.doubleclick.net/gampad/clk?id=121051231&iu=/4140/ostg.clktrk > > > > _______________________________________________ > Jump-pilot-devel mailing list > Jump-pilot-devel@lists.sourceforge.net > https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel > ------------------------------------------------------------------------------ Android apps run on BlackBerry 10 Introducing the new BlackBerry 10.2.1 Runtime for Android apps. Now with support for Jelly Bean, Bluetooth, Mapview and more. Get your Android app in front of a whole new audience. Start now. http://pubads.g.doubleclick.net/gampad/clk?id=124407151&iu=/4140/ostg.clktrk _______________________________________________ Jump-pilot-devel mailing list Jump-pilot-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/jump-pilot-devel
