RFR 8204310 : Simpler RandomAccessFile.setLength() on Windows

Ivan Gerasimov ivan.gerasimov at oracle.com
Mon Jun 18 18:57:24 UTC 2018


A gentle ping :)

Do you think it's good to go now?

With kind regards,

Ivan


On 6/10/18 11:15 PM, Ivan Gerasimov wrote:
> Hi Alan!
>
>
> On 6/6/18 6:57 AM, Alan Bateman wrote:
>> I think this should be okay but the Windows implementation has a long 
>> history of biting the fingers of anyone that dares touch it. 
>> Sometimes unexpected behavior changes only come to light long after 
>> the change. The recent mails here about Kafka and sparse files is a 
>> good example of that. So I think it's important to check the test 
>> coverage before pushing this change. Specifically I think we need to 
>> check that we have tests that
>>
>> 1. Exercise shrinking, extending, and not change the file length
>>
>> 2. Check getFilePointer when used with setLength.
>>
>> 3. Check how FileChannel behaves when created from a RandomAccessFile 
>> but the file length is changed after the FileChannel is obtained.
>>
> I extended the existing reg. test 
> java/io/RandomAccessFile/SetLength.java to cover more cases that 
> involve changing the file size.
>
> Also I added another regression test, as you Alan suggested, to check 
> that RandomAccessFile and its FileChannel behave consistently in 
> various scenarios.
>
> All the tests, including the new ones, pass on all supported platforms.
>
> BUGURL: https://bugs.openjdk.java.net/browse/JDK-8204310
> WEBREV: http://cr.openjdk.java.net/~igerasim/8204310/01/webrev/
>
> Would you please help review the fix?
>

-- 
With kind regards,
Ivan Gerasimov



More information about the core-libs-dev mailing list