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

Jack Li zheng.jun.li at oracle.com
Wed Oct 25 02:00:50 UTC 2017


Hi Lance,

The webrev is updated, can you please review it again?

JBS: https://bugs.openjdk.java.net/browse/JDK-8187954 <https://bugs.openjdk.java.net/browse/JDK-8187954> 
Webrev: http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/01 <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/01>

Summary of changes:

jaxws/src/java.xml.bind/share/classes/javax/xml/bind/*
JDK-8186946 - Fix accessibility and other issues in the java.xml.bind module

jaxws/src/java.xml.ws/share/classes/com/sun/xml/internal/messaging/saaj/**
JDK-8186314 - code at c.s.x.i.m.saaj.soap.MessageImpl must be modified to avoid crash after javac change
And also contains the fixes for importing nodes for SOAPDocumentFragment


Patch also contains several small bugfixes, not tracked in JBS.

> On Oct 11, 2017, at 18:47, Lance Andersen <lance.andersen at oracle.com> wrote:
> 
> Hi Jack,
> 
> I would prefer to see an updated webrev so that we do not inadvertently
> push these changes.
> 
> Best
> Lance
>> On Oct 11, 2017, at 3:26 AM, Jack Li <zheng.jun.li at oracle.com <mailto:zheng.jun.li at oracle.com>> wrote:
>> 
>> Hi Lance
>> 
>> I will update them in Metro repository, do I need to regenerate webrev?
>> or can you skip the files this time and I fix it in next integration?
>> 
>>> On Oct 9, 2017, at 19:35, Lance Andersen <lance.andersen at oracle.com <mailto:lance.andersen at oracle.com>> wrote:
>>> 
>>> Hi Jack,
>>> 
>>> UnMarshaller also has the same issue.  I would update the webrev given the number of places to help sanity check for omissions
>>> 
>>> Best
>>> Lance
>>>> On Oct 8, 2017, at 9:22 PM, Jack Li <zheng.jun.li at oracle.com <mailto:zheng.jun.li at oracle.com>> wrote:
>>>> 
>>>> Hi Lance,
>>>> 
>>>> the change is incorrect, it should be “javax/xml/bind”.
>>>> thanks a lot for your finding, do you think I need to fix it and resubmit the webrev this time?
>>>> or can you skip this file this time and I fix it in next integration?
>>>> 
>>>>> On Oct 4, 2017, at 02:09, Lance Andersen <Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>> wrote:
>>>>> 
>>>>> Hi Jack,
>>>>> 
>>>>> Is this change correct:
>>>>> 
>>>>> -------------
>>>>> --- old/src/java.xml.bind/share/classes/javax/xml/bind/Marshaller.java	2017-09-29 13:58:31.968185273 +0100
>>>>> +++ new/src/java.xml.bind/share/classes/javax/xml/bind/Marshaller.java	2017-09-29 13:58:31.676185267 +0100
>>>>> @@ -373,7 +373,7 @@
>>>>>       *      If the {@link ValidationEventHandler ValidationEventHandler}
>>>>>       *      returns false from its {@code handleEvent} method or the
>>>>>       *      {@code Marshaller} is unable to marshal {@code jaxbElement} (or any
>>>>> -     *      object reachable from {@code jaxbElement}).  See <a href="{@docRoot}/javax/xml/bind/Marshaller.html#elementMarshalling">
>>>>> +     *      object reachable from {@code jaxbElement}).  See <a href="{@docRoot}/java/xml/bind/Marshaller.html#elementMarshalling">
>>>>>       *      Marshalling a JAXB element</a>.
>>>>> 
>>>>> ------------
>>>>> 
>>>>> The URL that is being changed currently works
>>>>> 
>>>>> Best
>>>>> Lance
>>>>> On Sep 29, 2017, at 10:55 PM, Jack Li <zheng.jun.li at oracle.com <mailto:zheng.jun.li at oracle.com>> wrote:
>>>>> 
>>>>>> Hi,
>>>>>> 
>>>>>> Please review standalone JAXB/JAXWS changes, synced to jdk/jaxws repo.
>>>>>> 
>>>>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8187954 <https://bugs.openjdk.java.net/browse/JDK-8187954> <https://bugs.openjdk.java.net/browse/JDK-8187954 <https://bugs.openjdk.java.net/browse/JDK-8187954>>
>>>>>> Webrev: http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/ <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/> <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/ <http://cr.openjdk.java.net/~aefimov/jaxws-integrations/8187954/10/00/>>
>>>>>> 
>>>>>> Summary of changes:
>>>>>> 
>>>>>> jaxws/src/java.xml.bind/share/classes/javax/xml/bind/*
>>>>>> JDK-8186946 - Fix accessibility and other issues in the java.xml.bind module
>>>>>> 
>>>>>> jaxws/src/java.xml.ws/share/classes/com/sun/xml/internal/messaging/saaj/**
>>>>>> JDK-8186314 - code at c.s.x.i.m.saaj.soap.MessageImpl must be modified to avoid crash after javac change
>>>>>> And also contains the fixes for importing nodes for SOAPDocumentFragment
>>>>>> 
>>>>>> 
>>>>>> Patch also contains several small bugfixes, not tracked in JBS.
>>>>>> 
>>>>>> ----------------
>>>>>> Best regards
>>>>>> Jack Li
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> 
>>>>>> 
>>>>> 
>>>>> 
>>>>> <oracle_sig_logo.gif>
>>>>> 
>>>>> Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>>>> Oracle Java Engineering 
>>>>> 1 Network Drive 
>>>>> Burlington, MA 01803
>>>>>  <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>>> 
>>>> ----------------
>>>> Best regards
>>>> Jack Li
>>>> 
>>>> 
>>>> 
>>>> 
>>>> 
>>>> 
>>> 
>>> <oracle_sig_logo.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>  <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>>>  <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
>>> Oracle Java Engineering 
>>> 1 Network Drive 
>>> Burlington, MA 01803
>>> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
>>> 
>>> 
>>> 
>> 
>> 
>> ----------------
>> Best regards
>> Jack Li
> 
> <oracle_sig_logo.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>  <http://oracle.com/us/design/oracle-email-sig-198324.gif> <http://oracle.com/us/design/oracle-email-sig-198324.gif>
>  <http://oracle.com/us/design/oracle-email-sig-198324.gif>Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
> Oracle Java Engineering 
> 1 Network Drive 
> Burlington, MA 01803
> Lance.Andersen at oracle.com <mailto:Lance.Andersen at oracle.com>
> 
> 
> 


----------------
Best regards
Jack Li








More information about the core-libs-dev mailing list