RFR (JAXP) 8158084: Catalog API: JAXP XML Processor Support

[Joe Wang]

Thanks Lance!  Prabu believed that the JCK test should have been fixed 
by a previous patch. I'll send him a build to double-check.

Best,
Joe

On 7/27/16, 1:17 PM, Lance Andersen wrote:
> Hi Joe,
>
> This looks OK to me.
>
>
> You might want to get the JCK test addressed before pushing your changes
>
> Best
> Lance
>> On Jul 25, 2016, at 12:44 AM, huizhe wang <huizhe.wang at oracle.com 
>> <mailto:huizhe.wang at oracle.com>> wrote:
>>
>> Hi,
>>
>> This is an enhancement to support the Catalog API (javax.xml.catalog) 
>> throughout the JAXP processors. All of the CatalogFeatures are 
>> supported by the JAXP factories and processors (e.g. SAXParser), and 
>> corresponding System properties as specified in the CatalogFeatures. 
>> An additional javax.xml property "USE_CATALOG" is added to allow 
>> switching on/off the Catalog support for a processor or all of them 
>> (through the System properties or jaxp.properties).
>>
>> The USE_CATALOG feature is on by default for all processors. The only 
>> thing needed then, to resolve external references with a catalog, is 
>> setting the path to the catalog on the processor. While the 
>> USE_CATALOG is on by default, the Catalog support is noninterference. 
>> As long as no catalog is set, the Catalog is mute. Catalog, even if 
>> one exists, will not interfere with custom resolvers, it is simply 
>> ignored if a custom resolver is present.
>>
>>
>> specdiff: 
>> http://cr.openjdk.java.net/~joehw/jdk9/8158084/specdiff/overview-summary.html 
>> <http://cr.openjdk.java.net/%7Ejoehw/jdk9/8158084/specdiff/overview-summary.html>
>>
>> webrev: http://cr.openjdk.java.net/~joehw/jdk9/8158084/webrev/ 
>> <http://cr.openjdk.java.net/%7Ejoehw/jdk9/8158084/webrev/>
>>
>>
>> Tests: 140ish new test cases; all existing SQE/unit tests pass. JCK 
>> has a large number of failures due to one Catalog API test that left 
>> a configuration file behind. I'll discuss that with the JCK team.
>>
>>
>> Thanks,
>>
>> Joe
>>
>>
>>
>
> <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>
>
>
>