RFR: 8041626: [Event Request] Shutdown reason

[Robin Westberg]

Hi Erik,

> On 7 Feb 2018, at 21:47, Erik Gahlin <erik.gahlin at oracle.com> wrote:
> 
> Hi Robin,
> 
> I can sponsor this.
> 
> I wonder if we should change the name of the event to "Shutdown" instead? It will give us flexibility to add other shutdown related fields in the future.

Makes sense, I’ll rename it.

> Could you change the label and field to "Status Code" and statusCode.  Event fields should have headline-style capitalization and statusCode allows us to add other status related information in the future.

Removed the status field now, I’ll keep it in mind if it gets added back in the future.

Best regards,
Robin

> 
> Thanks
> Erik
> 
>> Hi all,
>> 
>> Please review the following change that adds an event-based tracing event that is generated when the VM shuts down. The intent is to capture shutdowns that occur after the VM has been properly initialized (as initialization problems would most likely mean that the tracing framework hasn’t been properly started either).
>> 
>> Issue: https://bugs.openjdk.java.net/browse/JDK-8041626 <https://bugs.openjdk.java.net/browse/JDK-8041626>
>> Webrev: http://cr.openjdk.java.net/~rwestberg/8041626/webrev.00/ <http://cr.openjdk.java.net/%7Erwestberg/8041626/webrev.00/>
>> Testing: hs-tier1,hs-tier2,jdk-tier1,jdk-tier2
>> 
>> Best regards,
>> Robin
>