[7u40] Request for phase 2 approval for 8016331 : Minor issues in event tracing metadata

[Erik Gahlin]

Seán Coffey skrev 2013-06-24 22:55:
> Erik,
>
>> It makes any GUI that displays the information look better (no red 
>> face).
> I'll assume the above was enough testing for this fix. 
Yes, looks good in UI.

> With the lack of a testcase, can you add the appropriate noreg-* label 
> to the bug report.
>
Yes, added noreg-trivial.

> Approved for jdk7u40. Work with Alejandro in case you want to push 
> this via the hsx integrations.
>

Thanks
Erik

> regards,
> Sean.
>
>
>
> On 24/06/13 15:17, Erik Gahlin wrote:
>> Hello,
>>
>> Could you please approve the following fix for 7u40:
>>
>> Bug:
>> http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=8016331
>>
>> Webrev:
>> http://cr.openjdk.java.net/~mgronlun/8016331/webrev01/
>>
>> Review:
>> http://mail.openjdk.java.net/pipermail/serviceability-dev/2013-June/010269.html 
>>
>>
>> Rationale:
>> Small changes to event tracing metadata, most importantly the split 
>> of the
>> ParallelScavangeHeapSummary event name. Very low risk, only changes 
>> to XML.
>> It makes any GUI that displays the information look better (no red 
>> face).
>>
>> Also, we want to avoid having different names for the same 
>> events/fields in
>> different releases, which would be the case if not fixed in 7u40.
>>
>> Thanks
>> Erik
>