G1 concurrent phase durations do not state the time units ("secs")

[Kirk Pepperdine]

Hi John,

No need for apologies for having a day off. We all need to do that. As others have said here, documentation on G1 is still quite sparse so I really appreciate your responses.

You can probably guess that I've looked at 100s of GC logs over the years from just about every version of the JVM there is. Can't say I've seen every combination of log thats out there but I've yet to see anything but K used as units. 
I just scanned the code for tty->print_cr... didn't see anything but a K format. Anyways, it not about the format. My only concern is the crossness in the measure.

As for the memory summary, it's not very important.. it just seems like a consistency issue but then I've not looked at enough G1 logs to know better.

Regards,
Kirk


On 2013-01-31, at 7:24 PM, John Cuthbertson <john.cuthbertson at oracle.com> wrote:

> Hi Kirk,
> 
> Apologies for the delay in getting back to you - I took yesterday afternoon off.
> 
> On 1/30/2013 12:25 PM, Kirk Pepperdine wrote:
>> Hi John,
>> 
>> Should I add that memory is reported in M,K, or B. I was really happy with just K being reported. Any comments on that?
> 
> And now I think there's 'G' as well.
> 
> I don't think I can comment on this, unfortunately. I always thought the GC logs did this unit conversion. At least I think I remember seeing it in the 1.4.2 time frame. I would be very nervous about making such a change since it affects all the collectors. Changing something like this would be controversial. There will be some, like you, who would welcome the change and there will be others who will complain bitterly. It's probably not going to happen soon, and we would need to make it switchable.
> 
> Cheers,
> 
> JohnC