> On Oct. 28, 2015, 6:30 p.m., Ashvin A wrote: > > build.gradle, line 385 > > <https://reviews.apache.org/r/39726/diff/1/?file=1111632#file1111632line385> > > > > Do we need to replace PermSize limit with Metaspace limit? I am not > > sure if limiting Metaspace is a good idea or not.
I think we should be OK. If that's not specified, the JVM should resize dynamically. We can always add that later if needed as well. - William ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/39726/#review104319 ----------------------------------------------------------- On Oct. 28, 2015, 6:19 p.m., William Markito wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/39726/ > ----------------------------------------------------------- > > (Updated Oct. 28, 2015, 6:19 p.m.) > > > Review request for geode, Darrel Schneider, Mark Bretl, and Dan Smith. > > > Bugs: GEODE-504 > https://issues.apache.org/jira/browse/GEODE-504 > > > Repository: geode > > > Description > ------- > > GEODE-504 - Removing MaxPermSize parameter > > > Diffs > ----- > > build.gradle 7e477f3 > gemfire-core/src/test/java/dunit/standalone/ProcessManager.java c4a55ed > > Diff: https://reviews.apache.org/r/39726/diff/ > > > Testing > ------- > > Local build. > Executed a few random DUnits manually. > > > Thanks, > > William Markito > >
