[jdk17u-dev] RFR: 8289524: Add JFR JIT restart event

[Thomas Stuefe]

On Thu, 3 Nov 2022 18:06:53 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

>> Having a couple of additional interesting (and not too complex !) events in the latest LTS is not a bad thing but this is my personal impression.  On the other hand I do not care that much, if it is rejected than probably all additional events would be rejected because the others I looked at that were interesting were more complex.
>> Our users will avoid non-LTS releases because of the effort and investment costs , so this means Sept 2023 + probably ~ one more year of testing etc. , until they can consume other interesting JFR events that came in after 17 , quite a long time I think.
>
>> Having a couple of additional interesting (and not too complex !) events in the latest LTS is not a bad thing but this is my personal impression. On the other hand I do not care that much, if it is rejected than probably all additional events would be rejected because the others I looked at that were interesting were more complex.
> 
> I think the bar should be high and there should be a good reason, for example to support new hardware or OS versions. Improve security.  Things that are essential to be able to use the existing JDK release.  I don't see this JFR event fall into that category.

@egahlin do you have any concrete concerns regarding the safety or risk of this patch? Or would it make the produced JFR output incompatible with JMC or other readers in any way?

If it is just a general concern about downporting practices, I vote for downporting it. As maintainers we have a high interest in keeping 17u stable, but we also need to have tools to analyze problems in the field.

-------------

PR: https://git.openjdk.org/jdk17u-dev/pull/853