[PATCH] Review Request for 8009258: TEST_BUG: java/io/pathNames/GeneralWin32.java fails intermittently

Eric Wang yiming.wang at oracle.com
Thu Mar 14 03:07:28 UTC 2013


Including more reviewer...

Thanks,
Eric
On 2013/3/13 14:28, Eric Wang wrote:
> Hi,
>
> Please review the code change, I have updated the test to make sure 
> test only access files and directories created by itself.
> http://cr.openjdk.java.net/~ewang/8009258/webrev.01/
>
> Here is the execution result:
> http://cr.openjdk.java.net/~ewang/8009258/GeneralWin32.jtr
>
> Thanks,
> Eric
> On 2013/3/5 1:39, Alan Bateman wrote:
>> On 04/03/2013 17:32, Eric Wang wrote:
>>> Hi,
>>>
>>> Please help to review fix below for bug 8009258 
>>> <https://jbs.oracle.com/bugs/browse/JDK-8009258>, 
>>> TEST_BUG:java/io/pathNames/GeneralWin32.java fails intermittently.
>>> http://cr.openjdk.java.net/~ewang/8009258/webrev.00/
>>>
>>> The File.canRead() method should not be used to check read 
>>> permission of a directory.
>>>
>>> Thanks,
>>> Eric
>> I wonder if it would be better to change this test so that it doesn't 
>> even attempt to poke around in these directories. I suggest this 
>> because there may be other activity going on at the same time. See 
>> also 8004096 where the test is running in agentvm mode and is 
>> straying into the directory used by another agent VM.
>>
>> -Alan
>




More information about the core-libs-dev mailing list