Alexander, Reviewed.
Thanks, Serguei On 10/21/16 06:25, Alexander Kulyakhtin wrote:
Hi Dmitry, Thank you very much for the review/ I've changed the CR title. Waiting for a (R)eviwer to approve this trivial one-line quarantine. Best regards, Alexander ----- Original Message ----- From: dmitry.dmitr...@oracle.com To: alexander.kulyakh...@oracle.com Cc: serviceability-dev@openjdk.java.net Sent: Friday, October 21, 2016 4:21:02 PM GMT +03:00 Iraq Subject: Re: RFR: 8168479: Backout the changes for JDK-8168478 Hi Alexander, can you please change bug title to "Quarantine AllModulesCommandTest.java test". The change looks good. Dmitry ----- Original Message ----- From: alexander.kulyakh...@oracle.com To: serviceability-dev@openjdk.java.net Sent: Friday, October 21, 2016 4:10:41 PM GMT +03:00 Iraq Subject: RFR: 8168479: Backout the changes for JDK-8168478 Hi, Could you, please, review this trivial change (quarantining a test). CR: https://bugs.openjdk.java.net/browse/JDK-8168479 "Backout the changes for JDK-8168478" Webrev: http://cr.openjdk.java.net/~akulyakh/8168479/ The test is being quarantining due to https://bugs.openjdk.java.net/browse/JDK-8168478 "JDWP: canRead() reports false for reading from the same module 'java.base'" (same as the previously unreproducible https://bugs.openjdk.java.net/browse/JDK-8166289) On the SQE side we do not currently see any issues with the test itself. Best regards, Alexander