RFR: 8176508 Update JAX-WS RI integration to latest version
Roman Grigoriadi
roman.grigoriadi at oracle.com
Fri Jun 2 05:29:33 UTC 2017
> On 1 Jun 2017, at 21:25, Mandy Chung <mandy.chung at oracle.com> wrote:
>
>
>> On May 31, 2017, at 5:06 AM, Roman Grigoriadi <roman.grigoriadi at oracle.com> wrote:
>>
>> Hi,
>>
>> New webrev can be found here:
>> http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8176508/02/ <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8176508/02/>
>
> jaxp/src/java.xml/share/classes/module-info.java
> I’m happy to see this qualified exports removed.
>
> Can you update jdk/test/jdk/modules/etc/JdkQualifiedExportTest.java to
> remove "java.xml/com.sun.xml.internal.stream.writers” from KNOWN_EXCEPTION.
>
> java.xml.ws/share/classes/com/sun/xml/internal/ws/api/streaming/XMLStreamReaderFactory.java
> java.xml.ws/share/classes/com/sun/xml/internal/ws/api/streaming/XMLStreamWriterFactory.java
> jaxws/src/java.xml.ws/share/classes/com/sun/xml/internal/ws/util/MrJarUtil.java
> MrJarUtil::getNoPoolProperty is not MR specific. Are you trying to keep
> the different default value when building for older release?
In standalone MrJarUtil exists and is packaged twice, but there is no need / way to sync JDK8 and below version of this file.
Here is second non-synced version of this file for 8 and below runtime:
https://github.com/javaee/metro-jax-ws/blob/master/jaxws-ri/rt/src/main/java/com/sun/xml/ws/util/MrJarUtil.java <https://github.com/javaee/metro-jax-ws/blob/master/jaxws-ri/rt/src/main/java/com/sun/xml/ws/util/MrJarUtil.java>
>
> It would be clearer if you want to define a constant for the default value that is subject to the runtime version.
>
> XMLStreamReaderFactory and XMLStreamWriterFactory would get the property value with the MR-specific default.
>
Than MrJarUtil would need to have only such constant for default value and metho getNoPoolProperty could be moved elsewhere.
> That’re the files I reviewed.
>
> Mandy
More information about the core-libs-dev
mailing list