JDK 9 RFR of JDK-8037278: sun/rmi/runtime/Log/6409194/NoConsoleOutput.java fails Intermittently: unexpected subprocess output (original) (raw)

Amy Lu amy.lu at oracle.com
Fri Nov 4 13:27:56 UTC 2016

Hi, Daniel

I updated webrev, removed the test dependency on TestLibrary. (RMI logging still be triggered thus no change to testing purpose.)

Please review again: http://cr.openjdk.java.net/~amlu/8037278/webrev.02/

Thanks, Amy

On 11/3/16 9:42 PM, Daniel Fuchs wrote:

On 03/11/16 13:35, Daniel Fuchs wrote:

Hi Amy,

Looks good to me. Maybe a followup should be logged to try and no longer call TestLibrary.getUnusedRandomPort(), as this the method that is causing the rogue output (and I see that it might print things to System.err as well). best regards, -- daniel

-- daniel On 03/11/16 13:22, Amy Lu wrote: Thank you Daniel for your review. Yes, agree and reverted those three lines. webrev updated: http://cr.openjdk.java.net/~amlu/8037278/webrev.01 Thanks, Amy On 11/3/16 8:37 PM, Daniel Fuchs wrote: Hi Amy,

I think it might still be useful to see what the subprocess prints on System.out in case of failure. I would leave lines 83, 86 and 87 unchanged. best regards, -- daniel On 03/11/16 11:58, Amy Lu wrote: Please review the patch for test sun/rmi/runtime/Log/6409194/NoConsoleOutput.java

bug: https://bugs.openjdk.java.net/browse/JDK-8037278 webrev: http://cr.openjdk.java.net/~amlu/8037278/webrev.00/ This test fails whenever there’s extra output from System.out (from the test application DoRMIStuff). But actually this test is for checking RMI logging messages and that are from System.err. Test should ignore the output from System.out. Thanks, Amy

More information about the core-libs-dev mailing list