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