[TEST-BUG] JDK 9 RFR of 8174127: (ch) Add instrumentation to java/nio/channels/FileChannel/Transfer.java

Alan Bateman Alan.Bateman at oracle.com
Thu Feb 9 11:31:34 UTC 2017


On 07/02/2017 22:22, Brian Burkhalter wrote:

> Please review at  your convenience.
>
> Issue:	https://bugs.openjdk.java.net/browse/JDK-8174127
> Patch:	http://cr.openjdk.java.net/~bpb/8174127/webrev.00/
>
> The purpose of this change is to instrument the test in question to provide information which might eventually lead to a fix for [1]. Debugging statements are added to the test to show which tests have been effected. Also, instead of exiting upon the first test failure, all tests are run and a failure count is kept with the overall test failing if that count is non-zero at the end. The test library utility also RandomFactory replaces the direct instantiation of Random to provide the ability to manage the randomness in the test.
>
>
I think it would be better to just convert it to a TestNG test. Also for 
the time out issue then I think there is enough in the original bug to 
see that it usually times out in Test08 which involves a 6GB file. It 
might be useful to put some output in that test so that you can see how 
long it takes to create the file. It might also be useful to increase 
the timeout on this test, just in case it is running close to the limit 
on some of these test machines.

-Alan.


More information about the nio-dev mailing list