RFR 8212807: tools/jar/multiRelease/Basic.java times out
Lance Andersen
lance.andersen at oracle.com
Thu May 30 20:29:28 UTC 2019
Hi Brent,
> On May 30, 2019, at 4:02 PM, Brent Christian <brent.christian at oracle.com> wrote:
>
> Hi, Lance
Thank you for the review.
>
> This change is to collect more information in case this happens again, yes?
This changes reduces the use of ProcessBuilder resulting in much improved test runs similar to what I did for:
https://mail.openjdk.java.net/pipermail/core-libs-dev/2019-April/059471.html <https://mail.openjdk.java.net/pipermail/core-libs-dev/2019-April/059471.html>
I took out the timing output from the tests after verifying the reduction in the test runs (which I ran 100 on the failing systems via mach 5 comparing before/after times)
Using ToolProvider is much more efficient than ProcessBuilder as I found out from the previous issue. The reduction in time was in line with the previous issue :-)
>
> Looks pretty good - just a couple comments:
>
>
> test/jdk/tools/jar/multiRelease/Basic.java
>
> 536 jar("ufm", jarfile, manifest.toString(),
>
> Is there a reason not to convert this to call jarTool() ?
Yes, java.util.jar.Attributes uses java.util.Logging to emit a warning for some reason which makes it a bit more difficult to deal with in this specific test. So I left this one test for now. At some point I want to go back through the other tests which use MRTestBase and convert the tests to also use ToolProvider and I can look to revisit the issue then.
Right now I am trying to cut down on the noise of some of the random timeouts :-)
>
> --
>
> test/jdk/tools/jar/multiRelease/MRTestBase.java
>
>
> L146-L152
>
> indentation looks off-by-one
>
Thank you for catching this, I updated the webrev : http://cr.openjdk.java.net/~lancea/8212807/webrev.01/ <http://cr.openjdk.java.net/~lancea/8212807/webrev.01/>
Best
Lance
>
> Thanks,
> -Brent
>
> On 5/30/19 9:21 AM, Lance Andersen wrote:
>> Hi all,
>> The following fix addresses an issue with an occasional timeout for tools/jar/multiRelease/Basic.java.
>> The webrev can be found at: http://cr.openjdk.java.net/~lancea/8212807/webrev.00/index.html <http://cr.openjdk.java.net/~lancea/8212807/webrev.00/index.html>
>> Best,
>> Lance
>> <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>
<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>
More information about the core-libs-dev
mailing list