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

[Christian Tornqvist]

> On Apr 1, 2019, at 8:55 AM, coleen.phillimore at oracle.com wrote:
> 
> +1.
> Are there other tests like this?  Seems to look like something one would copy and paste.

I’m sure there are other tests with the similar issues, this only became obvious because it’s done 10000 times in one test. I can look through the other uses of file URI’s and file issues for the ones that might be affected by this.

Thanks,
Christian

> Thanks,
> Coleen
> 
> On 4/1/19 11:48 AM, Harold Seigel wrote:
>> Hi Christian,
>> 
>> This looks good.
>> 
>> Thanks, Harold
>> 
>> On 4/1/2019 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/>
>>> 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?>
>