Review request for JDK-8134403: Nashorn react.js benchmark performance regression

[Sundararajan Athijegannathan]

Hi,

You could check out jdk8u-dev forest and build nashorn using "ant". You 
can get jdk8u forest from

     http://hg.openjdk.java.net/jdk8u/jdk8u-dev/

If you've complete jdk8u forest in directory $jdk8u

     cd $jdk8u/nashorn/make
     export JAVA_HOME=<recent_jdk8u_installation_home>
     ant clean jar

The resulting jar is produced @ $jdk8u/nashorn/dist directory. You can 
use that jar with jjs as

     jjs -J-Djava.ext.dirs=$jdk8u/nashorn/dist

NOTE: This is just a workaround for demonstration/talk purpose only. I 
don't recommend this as a production/deployment option - mixing binaries 
from local build + released version is problematic!

Hope this helps,
-Sundar

On 9/8/2015 7:10 PM, Sebastien Deleuze wrote:
> Hi,
>
> In my upcoming Nashorn + Spring talk next week, I would like to give people
> a feedback about Nashorn performances with real figures.
> I am going to bench my sample Nashorn + React.js application [1] and in
> order to show Nashorn as performant as it is, I need to use a build that
> contains JDK-8134403 fix.
>
> Sadly, only Java 8u66 seems to be available [3] for download.
> Any idea about how I could download a Java 8u72 MacOS build that contains
> JDK-8134403 fix or any news about backporting JDK-8134403 to Java 8u66 ?
>
> Thanks in advance for your feedback,
> Sébastien Deleuze
>
> [1] https://github.com/sdeleuze/spring-react-isomorphic
> [2] https://bugs.openjdk.java.net/browse/JDK-8134403
> [3] https://jdk8.java.net/download.html
>
>
> On Thu, Aug 27, 2015 at 10:03 PM, Martin Solli <msolli at gmail.com> wrote:
>
>> Much obliged!
>>
>> -martin
>>
>> tor. 27. aug. 2015 kl. 09.04 skrev Attila Szegedi <
>> attila.szegedi at oracle.com
>>> :
>>> I have just backported it to 8u-dev repository, which means that it
>> should
>>> definitely appear in 8u72 [1]. 8u66 [2] is already in RampDown 2 phase,
>>> meaning it only accepts critical updates at this time. I will file a
>>> critical update request for it, but it’s out of my hands whether it gets
>>> accepted for inclusion there.
>>>
>>> Attila.
>>>
>>> [1] http://openjdk.java.net/projects/jdk8u/releases/8u72.html
>>> [2] http://openjdk.java.net/projects/jdk8u/releases/8u66.html
>>>
>>>
>>>
>>> On Aug 27, 2015, at 8:56 AM, Martin Solli <msolli at gmail.com> wrote:
>>>
>>> Awesome! Any chance of getting this into a future Java 8 update?
>>> Performance is pretty dismal on u60 - can't get a medium-sized React
>>> component tree to render under 50 ms, whereas on u31 it renders in 1 ms
>>> (after ~7000 iterations).
>>>
>>> Thanks for all of your great work on Nashorn!
>>>
>>> -martin
>>>
>>> ons. 26. aug. 2015 kl. 15.41 skrev Attila Szegedi <
>>> attila.szegedi at oracle.com
>>>
>>> :
>>>
>>>
>>> Please review JDK-8134403 "Nashorn react.js benchmark performance
>>> regression" at <http://cr.openjdk.java.net/~attila/8134403/webrev.jdk9>
>>> for <https://bugs.openjdk.java.net/browse/JDK-8134403>
>>>
>>> Thanks,
>>>   Attila.
>>>
>>>
>>>