Code Review - CR# 7030573 insufficient disk space for large file test
Alan Bateman
Alan.Bateman at oracle.com
Fri Jan 6 20:32:59 UTC 2012
On 06/01/2012 19:47, Gary Adams wrote:
>
> I've updated the webrev with the temp file created in the current
> directory.
>
> I'm not sure what to do about the case if there is only a little space
> available
> only a small file will be used. Should the test fail and force the
> test operator
> to create a new test environment where 7G space is available?
>
> I lean toward allowing the test to pass using the space that is
> available.
I looked briefly at the updated webrev and I see it checks the usable
space on the file system/volume where the current directory is but
creates the file in the temporary directory, easily done :-)
I think you're right that the best we can do when there is insufficient
space is just to run with the small file. It doesn't test the original
issue but we don't want this test failing when there isn't sufficient space.
Rather than creating largefile, deleting it, and then creating it again
(twice) then how about:
File file = File.createTempFile("largefile", null, new File("."));
long usableSpace = largefile.getUsableSpace();
if (usuableSpace == 0L)
throw new RuntimeException(...);
long size = Math.min(usableSpace, 7405576182L);
recreateAsLargeFile(file, size);
-Alan.
More information about the core-libs-dev
mailing list