[14]RFR(XS): 8236062: Disable clhsdb initialization of SA javascript support since it will always fail, and will likely be removed soon

[Chris Plummer]

Hi Yasumasa,

Thanks for the review. If you mean plans for an alternate extension 
mechanism, we have none at the moment. We are of course open to 
suggestions. Some have been discussed on the recent thread regarding 
this topic, but there doesn't seem to be much consensus on how to 
approach this.

thanks,

Chris

On 12/16/19 11:36 PM, Yasumasa Suenaga wrote:
> Hi Chris,
>
> Looks good.
>
> BTW do you have any plan to provide alternative(s) on CLHSDB?
>
>
> Thanks,
>
> Yasumasa
>
>
> On 2019/12/17 14:36, Chris Plummer wrote:
>> Hello,
>>
>> Please review the following:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8236062
>> http://cr.openjdk.java.net/~cjplummer/8236062/webrev.00/
>>
>> Since SA javascript support is broken as described in [1] 
>> JDK-8235594, and we'll likely remove it, I'd like to at least get it 
>> disabled now. I'd like to get this into 14 mostly because I really 
>> want to get [2] JDK-8234048 fixed in 14 because we'll start seeing 
>> the clhsdb test failures on macos 10.14 and 10.15 more often over the 
>> coming months as we deploy more macosx test hosts with those 
>> versions. However, [2] JDK-8234048 is blocked by [3] JDK-8234277 
>> (which improves error checking and failure output for the clhsdb 
>> tests), and [3] JDK-8234277 is blocked by this CR because the 
>> exceptions produced when javascript fails to initialize end up 
>> cluttering the clhsdb test logs, even when the test passes (and is 
>> misleading when the test fails).
>>
>> Sorry about all the bug references and inter-dependencies. It's taken 
>> a while myself to get my head wrapped around how I wanted to approach 
>> fixing them all in a meaningful order.
>>
>> thanks,
>>
>> Chris
>>
>> [1] https://bugs.openjdk.java.net/browse/JDK-8235594
>> [2] https://bugs.openjdk.java.net/browse/JDK-8234048
>> [3] https://bugs.openjdk.java.net/browse/JDK-8234277