Re: RFR: 8158797: Test java/lang/management/MemoryMXBean/LowMemoryTest.java fails when GC is specified explicitly
Hi Sergey, Thank you very much for the review. Best regards, Alexander - Original Message - From: serguei.spit...@oracle.com To: alexander.kulyakh...@oracle.com, dmitry.samers...@oracle.com, dmitry.dmitr...@oracle.com Cc: serviceability-dev@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.dmitr...@oracle.com > To: alexander.kulyakh...@oracle.com, serviceability-dev@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
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.dmitr...@oracle.com To: alexander.kulyakh...@oracle.com, serviceability-dev@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
Re: RFR: 8158797: Test java/lang/management/MemoryMXBean/LowMemoryTest.java fails when GC is specified explicitly
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.dmitr...@oracle.com To: alexander.kulyakh...@oracle.com, serviceability-dev@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
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
RFR: 8158797: Test java/lang/management/MemoryMXBean/LowMemoryTest.java fails when GC is specified explicitly
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