RFR (XS) 8172972: Add gc/g1/logging/TestG1LoggingFailure.java to the ProblemList

classic Classic list List threaded Threaded
3 messages Options
Reply | Threaded
Open this post in threaded view
|

RFR (XS) 8172972: Add gc/g1/logging/TestG1LoggingFailure.java to the ProblemList

David Holmes
Subject says it all.

bug: https://bugs.openjdk.java.net/browse/JDK-8172972

Related/blocking bug: https://bugs.openjdk.java.net/browse/JDK-8169634

webrev: http://cr.openjdk.java.net/~dholmes/8172972/webrev/

Patch below for convenience

Thanks,
David
-----

--- old/test/ProblemList.txt 2017-01-18 20:08:23.481992142 -0500
+++ new/test/ProblemList.txt 2017-01-18 20:08:21.513878841 -0500
@@ -1,5 +1,5 @@
  #
-# Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
+# Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights
reserved.
  # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  #
  # This code is free software; you can redistribute it and/or modify it
@@ -55,6 +55,7 @@

  gc/g1/humongousObjects/objectGraphTest/TestObjectGraphAfterGC.java
8156755 generic-all
  gc/survivorAlignment/TestPromotionToSurvivor.java 8129886 generic-all
+gc/g1/logging/TestG1LoggingFailure.java 8169634 generic-all

 
#############################################################################
Reply | Threaded
Open this post in threaded view
|

Re: RFR (XS) 8172972: Add gc/g1/logging/TestG1LoggingFailure.java to the ProblemList

Kim Barrett
> On Jan 18, 2017, at 8:24 PM, David Holmes <[hidden email]> wrote:
>
> Subject says it all.
>
> bug: https://bugs.openjdk.java.net/browse/JDK-8172972
>
> Related/blocking bug: https://bugs.openjdk.java.net/browse/JDK-8169634
>
> webrev: http://cr.openjdk.java.net/~dholmes/8172972/webrev/
>
> Patch below for convenience
>
> Thanks,
> David
> -----
>
> --- old/test/ProblemList.txt 2017-01-18 20:08:23.481992142 -0500
> +++ new/test/ProblemList.txt 2017-01-18 20:08:21.513878841 -0500
> @@ -1,5 +1,5 @@
> #
> -# Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
> +# Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights reserved.
> # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
> #
> # This code is free software; you can redistribute it and/or modify it
> @@ -55,6 +55,7 @@
>
> gc/g1/humongousObjects/objectGraphTest/TestObjectGraphAfterGC.java 8156755 generic-all
> gc/survivorAlignment/TestPromotionToSurvivor.java 8129886 generic-all
> +gc/g1/logging/TestG1LoggingFailure.java 8169634 generic-all
>
> #############################################################################

Looks good.  Reviewed.

Also looks trivial.

Reply | Threaded
Open this post in threaded view
|

Re: RFR (XS) 8172972: Add gc/g1/logging/TestG1LoggingFailure.java to the ProblemList

David Holmes
Thanks Kim. Will push under trivial change rule.

David

On 20/01/2017 9:46 AM, Kim Barrett wrote:

>> On Jan 18, 2017, at 8:24 PM, David Holmes <[hidden email]> wrote:
>>
>> Subject says it all.
>>
>> bug: https://bugs.openjdk.java.net/browse/JDK-8172972
>>
>> Related/blocking bug: https://bugs.openjdk.java.net/browse/JDK-8169634
>>
>> webrev: http://cr.openjdk.java.net/~dholmes/8172972/webrev/
>>
>> Patch below for convenience
>>
>> Thanks,
>> David
>> -----
>>
>> --- old/test/ProblemList.txt 2017-01-18 20:08:23.481992142 -0500
>> +++ new/test/ProblemList.txt 2017-01-18 20:08:21.513878841 -0500
>> @@ -1,5 +1,5 @@
>> #
>> -# Copyright (c) 2016, Oracle and/or its affiliates. All rights reserved.
>> +# Copyright (c) 2016, 2017, Oracle and/or its affiliates. All rights reserved.
>> # DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
>> #
>> # This code is free software; you can redistribute it and/or modify it
>> @@ -55,6 +55,7 @@
>>
>> gc/g1/humongousObjects/objectGraphTest/TestObjectGraphAfterGC.java 8156755 generic-all
>> gc/survivorAlignment/TestPromotionToSurvivor.java 8129886 generic-all
>> +gc/g1/logging/TestG1LoggingFailure.java 8169634 generic-all
>>
>> #############################################################################
>
> Looks good.  Reviewed.
>
> Also looks trivial.
>