[11u] RFR: 8232806: Introduce a system property to disable eager lambda initialization

[Langer, Christoph]

Hi Jie,

backport looks good, thanks for doing this.

As for the CSR: It would not have been necessary as Oracle did that already for 11-pool: https://bugs.openjdk.java.net/browse/JDK-8233134. However, I also saw that your new CSR got approved (for their backport https://bugs.openjdk.java.net/browse/JDK-8233133)... So, now it's double safe ��

Cheers
Christoph


> -----Original Message-----
> From: jdk-updates-dev <jdk-updates-dev-bounces at openjdk.java.net> On
> Behalf Of Jie Kang
> Sent: Mittwoch, 18. Dezember 2019 14:50
> To: jdk-updates-dev <jdk-updates-dev at openjdk.java.net>
> Subject: [11u] RFR: 8232806: Introduce a system property to disable eager
> lambda initialization
> 
> Hi,
> 
> Please review this backport for JDK-8232806. This is useful for
> graalvm and has also already been backported into 11.0.7-oracle, so
> should be included at least for parity.
> 
> Backport Webrev
> http://cr.openjdk.java.net/~jkang/JDK-8232806/webrev.1/
> 
> Original Commit
> https://hg.openjdk.java.net/jdk/jdk/rev/27c2d2a4b695
> Original Bug
> https://bugs.openjdk.java.net/browse/JDK-8232806
> 
> The patch did not function cleanly due to missing
> GetBooleanAction.privilegedGetProperty in 11u. The call to this has
> been replaced by the lengthier form used elsewhere and also prior to
> the introduction of above static helper method:
> 
> disableEagerInitialization = AccessController.doPrivileged(
>     new GetBooleanAction(disableEagerInitializationKey)).booleanValue();
> 
> Let me know what you think about this adjustment. The tier one tests
> including the modified LambdaTest6 passed for me.
> 
> 
> Regards,