RFR: 8164479: Update JAX-WS RI integration to latest version

Roman Grigoriadi roman.grigoriadi at oracle.com
Mon Sep 26 13:07:08 UTC 2016


Hi,

Thank you for review. We were discussing java.net will go down 
eventually, links to jaxb.java.net is best we have now (to fix submitted 
javadoc bug), we will update links again when we know new location of 
standalone projects documentation.

Regarding clean standalone TCK tests - yes they are clean against these 
changes, but there is one caveat to it:
Standalone tests were never run on JDK9 yet. We are trying to utilize a 
"multirelease-jar" feauture of JDK9 in standalone projects, to avoid 
using reflection for JDK9 api and checks for module system. This 
multirelease version of standalone project is still work in progress, 
tests cannot be run against it yet. We are running standalone TCK tests 
on JDK8 in separate branch, which has all these changes merged, but 
lacks JDK9 only compatible classes - namely catalog API and some 
resource loading with Module#getResourceAsStream. So lets say TCK tests 
are "almost clean" against this version.

Roman

On 09/26/2016 09:25 AM, Alan Bateman wrote:
>
>
> On 26/09/2016 00:41, Aleks Efimov wrote:
>> Hi Alan, Joe, Mandy, Roman,
>>
>> Suggested changes to the comment section (will bring this change to 
>> standalone JAXB) and to the exported internal API were made:
>> com.sun.xml.internal.stream.writers, 
>> com.sun.org.apache.xml.internal.resolver, 
>> com.sun.org.apache.xml.internal.resolver.tools dependencies were 
>> removed.
>>
>> The updated webrev:
>> http://cr.openjdk.java.net/~aefimov/8164479/01
> Thanks Aleks.
>
> If com.sun.xml.internal.stream.writers is not used directly then I 
> assume the comment "// reflection access ..." can be removed too.
>
> Do we have any issues tracking the issue of direct access to types in 
> com.sun.org.apache.xerces.internal.**. Those will need to be addressed 
> if JAX-WS is to be truely standalone without needing --add-exports 
> options on the command line.
>
> For java.activation then I assume you can run deps too as it should no 
> longer need the `requires java.desktop`. Also I think we can improve 
> the javadoc of CommandInfo.getCommandObject to replace the proposed 
> "If the current runtime environment supports Beans.instantiate ..." to 
> something like "If java.beans.Beans is visible ...".
>
> -Alan
>



More information about the core-libs-dev mailing list