Hi, Lance

This change is to collect more information in case this happens again, yes?

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() ?

--

test/jdk/tools/jar/multiRelease/MRTestBase.java


L146-L152

indentation looks off-by-one


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.ander...@oracle.com <mailto:lance.ander...@oracle.com>



Reply via email to