Review request JDK-8068242: quarantine the test IsModifiableClassAgent.java until JDK-8068162 is resolved

Dmitry Samersoff dmitry.samersoff at oracle.com
Thu Dec 25 13:00:27 UTC 2014


I'll push it later today.

On 2014-12-25 15:45, Alexander Kulyakhtin wrote:
> Hi,
> 
> Could someone, please, push the  fix reviewed below?
> 
> This is just one-line change to quarantine a test while the fix is being
> worked on.
> 
> bug: https://bugs.openjdk.java.net/browse/JDK-8068242
> webrev:
> http://cr.openjdk.java.net/~eistepan/~akulyakhtin/8068242/webrev.00
> <http://cr.openjdk.java.net/%7Eeistepan/%7Eakulyakhtin/8068242/webrev.00>/
> 
> Thank you very much,
> Alexander
> 
> ----- Original Message -----
> From: jaroslav.bachorik at oracle.com
> To: dmitry.samersoff at oracle.com, serviceability-dev at openjdk.java.net,
> alexander.kulyakhtin at oracle.com
> Sent: Wednesday, December 24, 2014 11:07:58 PM GMT +04:00 Abu Dhabi / Muscat
> Subject: Re: Review request JDK-8068242: quarantine  the
> test        IsModifiableClassAgent.java until JDK-8068162 is resolved
> 
> Reviewed.
> 
> Merry Christmas, ho-ho-ho :)
> 
> On 24 December 2014 17:11:15 CET, dmitry.samersoff at oracle.com wrote:
> 
>     Looks good for me (not a reviewer).
> 
>     --Dmitry
> 
> 
> 
>     -----Original Message-----
>     From: Alexander Kulyakhtin <alexander.kulyakhtin at oracle.com>
>     To: serviceability-dev at openjdk.java.net
>     Cc: "Dmitry Михайлович Samersoff" <dmitry.samersoff at oracle.com>,
>     Serguei Vladimirovich Spitsyn <serguei.spitsyn at oracle.com>
>     Sent: Wed, 24 Dec 2014 17:32
>     Subject: Review request JDK-8068242: quarantine the test
>     IsModifiableClassAgent.java until JDK-8068162 is resolved
> 
>     Hi,
> 
>     Could I, please, have a review of this fix. This is a oneline change
>     to quarantine a test until JDK-8068162 <about:blank> is resolved
> 
>     bug: https://bugs.openjdk.java.net/browse/JDK-8068242
>     webrev:
>     http://cr.openjdk.java.net/~eistepan/~akulyakhtin/8068242/webrev.00/
> 
>     @ignore JDK-8068162 <about:blank> added to the test to have it
>     ignored until JDK-8068162 <about:blank> is resolved
> 
>     Thanks,
>     Alexander
> 
> 
> -- 
> Sent from my Android device with K-9 Mail. Please excuse my brevity.


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.


More information about the serviceability-dev mailing list