RFR: 8150145: javax/xml/jaxp/unittest/common/TransformationWarningsTest.java and ValidationWarningsTest.java failed intermittently without any error message
huizhe.wang at oracle.com
Wed Aug 31 18:39:25 UTC 2016
On 8/31/16, 10:51 AM, Aleks Efimov wrote:
> Hi Joe,
> Thank you for reviewing the changes. I found one more inconsistency
> with these tests:
> The TestSAXDriver class is not compiled by default now and because of
> that the default SAX driver were used and the original issue was not
> reproduced properly. New webrev can be found here:
> About intermittent failure: The test was added with 'othervm'
> initially, but few months ago the transformation test was failing due
> to concurrency issue in PackageEntry::package_exports_do (
> JDK-8152404). Now failures are not reproducible with latest JDK9
> builds and latest version of the tests.
Did you mean it was later changed to othervm? The initial check-in
record shows it wasn't 
othervm provents the potential interference by other tests that also set
the system property, and therefore removes the need for a further
refactoring of the code. So this is not required, but you may remove the
SAX driver setting and use a new instance instead, e.g.
SAXSource saxSource = new SAXSource(new TestSAXDriver(), new
> On 31/08/16 19:25, Joe Wang wrote:
>> Hi Aleksej,
>> It's good to put the tests back online. Thanks for the diligent work!
>> I believe the change that made the tests run in othervm could have
>> fixed the intermittent issue. But adding debugging code can always be
>> helpful in case of failures.
>> On 8/30/16, 12:01 PM, Aleks Efimov wrote:
>>> Please, help to review the tests fix.
>>> Webrev: http://cr.openjdk.java.net/~aefimov/8150145/00/
>>> JBS: https://bugs.openjdk.java.net/browse/JDK-8150145
>>> The list of changes:
>>> 1. Two tests were modified to print exception occurred during
>>> execution. Before that these tests were failing silently.
>>> 2. The tests were slightly modified to correctly run with security
>>> manager run mode recently added to JAXP tests (JDK-8067170).
>>> 3. TransformationWarningsTest were modified to synchronize the
>>> TransformerFactory instantiation.
>>> The modified tests were executed 2000+ times (alongside to other
>>> tests from jaxp/test/javax/xml/jaxp/unittest/common) on linux-x64
>>> and there were no failures observed. According to this result and
>>> that the tests were modified not to fail silently. I would like to
>>> remove these two tests from the jaxp problem list.
>>> If the tests will continue to fail on some configurations (JPRT
>>> shows no failures for few runs though) the proposed changes will
>>> help to diagnose the cause of failures.
>>> With Best Regards,
More information about the core-libs-dev