[8u] RFR: 8238164: Update Apache Xerces to version 2.12.0
Elliott Baron
ebaron at redhat.com
Wed Jul 21 17:32:35 UTC 2021
Hi Andrew,
On 2021-07-21 1:14 p.m., Andrew Hughes wrote:
> On 17:35 Mon 19 Jul , Elliott Baron wrote:
>> Hi,
>>
>> I'd like to request a review of 8238164 for 8u for parity with Oracle JDK
>> 8u261.
>>
>> 8238164 is an 8-only bug, but has an extensive "relates to" list that I used
>> as a guide for what to backport. I had considered copying over Xerces 2.12.0
>> en masse from JDK 11, but many of the related changes in 8238164 extend
>> beyond the Xerces package (e.g. Xalan, XML).
>>
>> The chief concern I have with this backport is the lack of unit tests in the
>> JAXP repo in 8u. Some of the referenced fixes below include new tests or
>> changes to tests that were added in JDK 9 beginning with "8043084: XML JAXP
>> unittest co-location".
>>
>> I'd also like to note that there are minor changes to
>> javax/xml/stream/XMLEventReader and javax/xml/stream/XMLStreamReader. The
>> former adds an @Override tag, the other changes are all to documentation.
>> I'm not sure if the override tag poses a problem to API compatibility.
>>
>> One other minor issue is the @deprecated tag added to Xerces' internal
>> serializer. This is not public API, but I was unsure whether to change the
>> version from JDK 1.9 to JDK 1.8. I've just left it alone in this initial
>> webrev. See:
>> com/sun/org/apache/xml/internal/serialize/BaseMarkupSerializer.java.
>>
>> The end of this mail contains a detailed list of related fixes mentioned in
>> 8238164 that are included in this webrev, along with what modifications I
>> made for them to apply to the 8u tree. The list also covers which of related
>> fixes are not included in this webrev, and why not.
>>
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8238164
>> Webrev: https://cr.openjdk.java.net/~ebaron/jdk8u/JDK-8238164/webrev.00/
>>
>> Testing: x86_64 build, jdk_tier1, jdk_other (includes javax/xml) tests
>>
>> Thanks,
>> Elliott
>>
>> ---
>
> Are you saying this webrev is all (or most of) the fixes you listed merged
> together?
>
> I don't know how to even begin reviewing such a thing. Can we not just
> backport the fixes individually, so they can be easily reviewed?
>
Yes, this is an all-in-one webrev. 8238164 led me to believe this was
the approach taken by Oracle, just based on the information in JBS. The
individual fixes do not show as being backported to Oracle JDK 8.
Nevertheless, it's no problem to propose these fixes one at a time. I
understand it will be easier to review that way.
Thanks,
Elliott
More information about the jdk8u-dev
mailing list