RFR: 8214799: Add package declaration to each JTREG test case in the gc folder

Thomas Schatzl thomas.schatzl at oracle.com
Wed Jan 23 09:51:54 UTC 2019


Hi,

On Thu, 2019-01-17 at 16:58 +0100, Leo Korinth wrote:
> Hi,
> 
> I have added package name to newly added
> TestPeriodicLogMessages.java.
> 
> I would be happy for a review from a reviewer (for the whole change,
> not 
> only this test).
> 
> The updated test case has been tested.

Good change to decrease the threshold to improve these :)

.02 changes and below diff look good.

Thomas

> 
> /Leo
> 
> 
> diff --git a/test/hotspot/jtreg/gc/g1/TestPeriodicLogMessages.java 
> b/test/hotspot/jtreg/gc/g1/TestPeriodicLogMessages.java
> index 2cede583c45..bd02a6e1c0a 100644
> --- a/test/hotspot/jtreg/gc/g1/TestPeriodicLogMessages.java
> +++ b/test/hotspot/jtreg/gc/g1/TestPeriodicLogMessages.java
> @@ -21,6 +21,8 @@
>    * questions.
>    */
> 
> +package gc.g1;
> +
>   /**
>    * @test TestPeriodicLogMessages
>    * @bug 8216490
> @@ -29,6 +31,7 @@
>    * @library /test/lib /
>    * @modules java.base/jdk.internal.misc
>    * @modules java.management/sun.management
> + * @run main gc.g1.TestPeriodicLogMessages
>    */
> 
>   import jdk.test.lib.process.OutputAnalyzer;
> 




More information about the hotspot-gc-dev mailing list