RFR: 8212627: [TESTBUG] runtime/CreateMirror/ArraysNewInstanceBug.java timed out

[Daniel D. Daugherty]

On 4/1/19 11:43 AM, Christian Tornqvist wrote:
> Hi,
>
> Please review this small test fix for an issue where ArraysNewInstanceBug.java times out on certain Windows machines. The test uses a file URI to point to where to find a test class, this URI is supposed to be pointing to the local file system but the file URI uses the format of a network location (two instead of three forward slashes [1]). This causes network lookups on Windows and, depending on the network configuration, this might take some time and cause the test to timeout.
>
> Change has been tested by running the test in Mach 5 on Linux, Mac, Solaris and Windows.
>
> Webrev: http://cr.openjdk.java.net/~ctornqvi/webrev/8212627/webrev.00/ <http://cr.openjdk.java.net/~ctornqvi/webrev/8212627/webrev.00/>

test/hotspot/jtreg/runtime/CreateMirror/ArraysNewInstanceBug.java
     I like the use of toURI() and toURL() rather than trying to
     manually construct a URL that follows the rules.

Thumbs up.

Dan


> Bug: https://bugs.openjdk.java.net/browse/JDK-8212627 <https://bugs.openjdk.java.net/browse/JDK-8212627>
>
> Thanks,
> Christian
>
> [1] https://en.wikipedia.org/wiki/File_URI_scheme#How_many_slashes? <https://en.wikipedia.org/wiki/File_URI_scheme#How_many_slashes?>