[PATCH] Review request: 6962637 TEST_BUG: java/io/File/MaxPathLength.java may fail in busy system

Amy Lu amy.lu at oracle.com
Wed Aug 22 09:07:25 UTC 2012

Hi, Eric

I'm a little confused about on which platform should this test run?

In the old code, the test should run:
     i = 4,5,6 on all platforms except win_9 win_me ( and solaris due to 
machine issue(?))
     i = 20 on windows only
Now in new code, the whole test will only run on windows.


On 8/22/12 4:10 PM, Eric Wang wrote:
> Hi David,
> To use dirFile.length() == 0 to replace the disk space measurement as 
> guess the original test tries to prove that there's no extra disk 
> spent when creating empty folders.
> Regards,
> Eric
> On 2012/8/22 16:02, David Holmes wrote:
>> Hi Eric,
>> On 22/08/2012 5:54 PM, Eric Wang wrote:
>>> Please help to review the fix below for bug **6962637
>>> <http://monaco.us.oracle.com/detail.jsf?cr=6962637> TEST_BUG:
>>> java/io/File/MaxPathLength.java may fail in busy system
>>> http://dl.dropbox.com/u/90659131/fixes/6962637/webrev/index.html
>>> Root cause is the test is not reliable when system is busy, if disk
>>> space is changed by some other program, test can fail. so the fix is to
>>> check file.length() instead of disk space.
>> I can not figure out what the original version of the test was doing 
>> with the disk space measurements but I can certainly appreciate that 
>> they could be changing dynamically while the test is running.
>> But I don't really understand why dirFile is expected to have a zero 
>> length ???
>> David
>>> Regards,
>>> Eric

More information about the core-libs-dev mailing list