This looks fine Aleksei.  Approved for jdk8u-dev.

The JDK-8159240 record is currently closed as a duplicate. I'd suggest you reopen that master record for your 8u-dev port and mark it 9-na.

regards,
Sean.

On 27/09/2017 22:28, Aleks Efimov wrote:
Hi,

Can I, please, ask for review and approval of JDK-8159240 backport to JDK8? This bug was resolved in JDK9 as part of JAXWS-RI sync changeset (JDK-8164479) therefore separate review for the partially backported changes is needed: http://cr.openjdk.java.net/~aefimov/8159240/8/00/

File changes partially backported to JDK8 from JDK9 sync changeset:
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l37.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l38.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l39.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l40.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l41.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l42.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l43.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l44.1
http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052#l45.1

Testing:
    New regression test and all jaxb/ws related JCK/JTREG tests shows no failures.

JBS bug:
    https://bugs.openjdk.java.net/browse/JDK-8159240

JAXWS-RI sync bug:
    https://bugs.openjdk.java.net/browse/JDK-8164479

JDK9 JAXWS-RI sync changeset:
    http://hg.openjdk.java.net/jdk9/dev/jaxws/rev/26c9b9c51052

JDK9 JAXWS-RI sync review threads:
http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-September/043724.html http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-October/043920.html http://mail.openjdk.java.net/pipermail/core-libs-dev/2016-November/044513.html

JAXWS-RI changes for XSOM parser:
https://github.com/javaee/jaxb-v2/commit/cc9c09a71d739989d1e250d9db8c1e011215abfd#diff-796464168dc0014203b00c9728d41de8

With Best Regards,
Aleksei


Reply via email to