RFR: 8158797: Test java/lang/management/MemoryMXBean/LowMemoryTest.java fails when GC is specified explicitly

[Alexander Kulyakhtin]

Hi Sergey,

Thank you very much for the review.

Best regards,
Alexander

----- Original Message -----
From: serguei.spitsyn at oracle.com
To: alexander.kulyakhtin at oracle.com, dmitry.samersoff at oracle.com, dmitry.dmitriev at oracle.com
Cc: serviceability-dev at openjdk.java.net
Sent: Thursday, October 13, 2016 3:40:37 PM GMT +03:00 Iraq
Subject: Re: RFR: 8158797: Test java/lang/management/MemoryMXBean/LowMemoryTest.java fails when GC is specified explicitly

Alexander,

It looks good to me.

Thanks,
Serguei


On 10/13/16 05:37, Alexander Kulyakhtin wrote:
> Dmitry,
>
> Thank you very much for the review.
>
> I still need a (R)eveiwer to approve the fix. Could someone, please, review this simple one-line change?
>
> Best regards,
> Alexander
>
> ----- Original Message -----
> From: dmitry.dmitriev at oracle.com
> To: alexander.kulyakhtin at oracle.com, serviceability-dev at openjdk.java.net
> Sent: Wednesday, October 12, 2016 5:33:45 PM GMT +03:00 Iraq
> Subject: Re: RFR: 8158797: Test java/lang/management/MemoryMXBean/LowMemoryTest.java fails when GC is specified explicitly
>
> Hi Alexander,
>
> Looks good. Please correct copyright year in the header(2015 -> 2016).
> Not need a new webrev for that.
>
> Thanks,
> Dmitry
>
> On 10.10.2016 13:51, Alexander Kulyakhtin wrote:
>> Hi,
>>
>> Could you, please, review this simple, test-only change:
>>
>> CR: https://bugs.openjdk.java.net/browse/JDK-8158797 "Test java/lang/management/MemoryMXBean/LowMemoryTest.java fails when GC is specified explicitly"
>> Webrev: http://cr.openjdk.java.net/~akulyakh/8158797/test/java/lang/management/MemoryMXBean/LowMemoryTest.java.udiff.html
>>
>> In order to verify some expected behavior, the test specifies its own GC options when running.
>> Currently, if the testing framework runs this test with some other explicitly specifed GC options, then the test fails with "Conflicting collector combinations in option list" message.
>> We are modifying the test so that it runs only when the framework does not specify any GC options.
>>
>> Best regards,
>> Alexander