Re: Review Request: RT-39975 - AppCDS support for packager

2015-03-31 Thread Mike Hearn
The bug is restricted - intentional? I'm guessing this is class data sharing and would make startup of packaged apps faster? On Tue, Mar 31, 2015 at 12:52 AM, Chris Bensen chris.ben...@oracle.com wrote: +1 On Mar 30, 2015, at 3:11 PM, Danno Ferrin danno.fer...@oracle.com wrote: Kevin,

Re: Review Request: RT-39975 - AppCDS support for packager

2015-03-31 Thread Chris Bensen
Correct! On Mar 31, 2015, at 4:41 AM, Mike Hearn m...@plan99.net wrote: The bug is restricted - intentional? I'm guessing this is class data sharing and would make startup of packaged apps faster? On Tue, Mar 31, 2015 at 12:52 AM, Chris Bensen chris.ben...@oracle.com wrote: +1 On

Re: Review Request: RT-39975 - AppCDS support for packager

2015-03-31 Thread Mike Hearn
Do you have any stats on the perf improvement? My understanding of CDS is that it was primarily meant to reduce memory usage on systems where multiple Java apps are running on the same JRE simultaneously. I guess that won't apply to packaged apps so the only benefit can be startup time. On Tue,

Re: Review Request: RT-39975 - AppCDS support for packager

2015-03-31 Thread Chris Bensen
No, I don’t have any stats to share about the performance benefits, sorry. On Mar 31, 2015, at 1:42 PM, Mike Hearn m...@plan99.net wrote: Do you have any stats on the perf improvement? My understanding of CDS is that it was primarily meant to reduce memory usage on systems where multiple

Review Request: RT-39975 - AppCDS support for packager

2015-03-30 Thread Danno Ferrin
Kevin, Chris, please review jira: https://javafx-jira.kenai.com/browse/RT-39975 webrev: http://cr.openjdk.java.net/~shemnon/RT-39975/webrev.00/

Re: Review Request: RT-39975 - AppCDS support for packager

2015-03-30 Thread Chris Bensen
+1 On Mar 30, 2015, at 3:11 PM, Danno Ferrin danno.fer...@oracle.com wrote: Kevin, Chris, please review jira: https://javafx-jira.kenai.com/browse/RT-39975 webrev: http://cr.openjdk.java.net/~shemnon/RT-39975/webrev.00/