[JFR incubator] [RFR] JDK-8238590: Enable JFR by default during compilation in 8u

[Mario Torre]

On Tue, Feb 18, 2020 at 2:11 PM Ben Evans <bevans at newrelic.com> wrote:

>  I completely agree. I have anecdotal reports of performance regression with the JFR code compiled in, but those are against the incubator repo so it's not apples-apples.
>
> Having the JFR code in mainline, disabled by default but capable of manual enablement will greatly simplify the process of getting properly accurate feedback - and will hopefully mean broader testing from a wider range of users as well.
>
> I'm very, very keen to get JFR available for everyone using JDK 8, but in my opinion a few months delay is greatly preferable to the risk of a performance regression that could affect people's ability to upgrade to 8u262+

Hi Ben,

I agree but we need some actual numbers on an apple-to-apple
comparison to properly assess the cost, otherwise we risk sitting with
the patch a few months without solving the underlying issue. Do you
have something that is more than just anecdotal to share?

Thanks!

Cheers,
Mario

-- 
Mario Torre
Associate Manager, Software Engineering
Red Hat GmbH <https://www.redhat.com>
9704 A60C B4BE A8B8 0F30  9205 5D7E 4952 3F65 7898