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