RFR: JDK-8068589: GCCause should distinguish jcmd GC.run from System.gc()

[Yasumasa Suenaga]

Hi,

I need more reviewer.
Could you review it?

http://cr.openjdk.java.net/~ysuenaga/JDK-8068589/webrev.02/

Thanks,

Yasumasa
2015/01/28 17:24 "Staffan Larsen" <staffan.larsen at oracle.com>:

> Looks good!
>
> Thanks,
> /Staffan
>
> > On 28 jan 2015, at 05:48, Yasumasa Suenaga <yasuenag at gmail.com> wrote:
> >
> > Hi Staffan, Kirk,
> >
> > I agree to set "Diagnostic Command" to GCCause.
> > So I applied it to new patch.
> >
> > http://cr.openjdk.java.net/~ysuenaga/JDK-8068589/webrev.02/
> >
> > Could you review it again?
> >
> >
> > Thanks,
> >
> > Yasumasa
> >
> >
> > On 2015/01/28 5:06, Kirk Pepperdine wrote:
> >> Hi Staffan,
> >>
> >>>>
> >>>> Anyway, it’s a record in a GC log so I don’t see the value of GC.run.
> Certainly “DiagCmd" or even "Diagnostic Command” seems sufficient given the
> context.
> >>>
> >>> Let’s go with “Diagnostic Command”, then.
> >>
> >> Thank you!
> >>
> >> Regards,
> >> Kirk
> >>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/serviceability-dev/attachments/20150203/8db8fff9/attachment.html>