RFR(XS) : 8190890 : remove hotspot_tier1_{compiler,gc}_closed groups

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

RFR(XS) : 8190890 : remove hotspot_tier1_{compiler,gc}_closed groups

Igor Ignatyev
http://cr.openjdk.java.net/~iignatyev//8190890/webrev.00/index.html
> 9 lines changed: 0 ins; 9 del;

Hi all,

could you please review this tiny patch which removes unneeded jtreg test groups?

webrev: http://cr.openjdk.java.net/~iignatyev//8190890/webrev.00/index.html
JBS: https://bugs.openjdk.java.net/browse/JDK-8190890

Thanks,
-- Igor

Reply | Threaded
Open this post in threaded view
|

Re: RFR(XS) : 8190890 : remove hotspot_tier1_{compiler,gc}_closed groups

Erik Helin-2
On 11/08/2017 01:56 AM, Igor Ignatyev wrote:
> http://cr.openjdk.java.net/~iignatyev//8190890/webrev.00/index.html
>> 9 lines changed: 0 ins; 9 del;
>
> Hi all,
>
> could you please review this tiny patch which removes unneeded jtreg test groups?
>
> webrev: http://cr.openjdk.java.net/~iignatyev//8190890/webrev.00/index.html
> JBS: https://bugs.openjdk.java.net/browse/JDK-8190890

Looks good, Reviewed. I would say that this change classifies as
trivial, so you only need one Reviewer. Thanks for cleaning this up.

Thanks,
Erik

> Thanks,
> -- Igor
>
Reply | Threaded
Open this post in threaded view
|

Re: RFR(XS) : 8190890 : remove hotspot_tier1_{compiler,gc}_closed groups

Igor Ignatyev
Thanks Erik.
-- Igor

> On Nov 8, 2017, at 9:16 AM, Erik Helin <[hidden email]> wrote:
>
> On 11/08/2017 01:56 AM, Igor Ignatyev wrote:
>> http://cr.openjdk.java.net/~iignatyev//8190890/webrev.00/index.html
>>> 9 lines changed: 0 ins; 9 del;
>> Hi all,
>> could you please review this tiny patch which removes unneeded jtreg test groups?
>> webrev: http://cr.openjdk.java.net/~iignatyev//8190890/webrev.00/index.html
>> JBS: https://bugs.openjdk.java.net/browse/JDK-8190890
>
> Looks good, Reviewed. I would say that this change classifies as trivial, so you only need one Reviewer. Thanks for cleaning this up.
>
> Thanks,
> Erik
>
>> Thanks,
>> -- Igor