Proposal for back-porting JFR to OpenJDK8u

Andrey Petushkov andrey at azul.com
Fri Feb 15 16:22:17 UTC 2019


[fixed subject, removed jfr-dev maillist]

In the meanwhile I’ve updated webrev with a backports of JDK-8207392 (JFR profiling for PPC) and shared part of (necessary for the latter) JDK-8159284

Regards,
Andrey

On 14 Feb 2019, at 14:20, Mario Torre <neugens at redhat.com<mailto:neugens at redhat.com>> wrote:



On Tue, Feb 12, 2019 at 12:41 PM Andrew Haley <aph at redhat.com<mailto:aph at redhat.com>> wrote:
On 2/11/19 1:01 PM, Mario Torre wrote:
> On Mon, Feb 11, 2019 at 12:29 PM Severin Gehwolf <sgehwolf at redhat.com<mailto:sgehwolf at redhat.com>> wrote:
>>
>> On Fri, 2019-02-08 at 19:00 +0100, Mario Torre wrote:
>>> Andrew, do you think we can have a repository created for this
>>> purpose?
>>
>> At the OpenJDK Committers Workshop a JDK 8u sandbox repository was
>> discussed. Perhaps that would fit the bill for this backport in a more
>> generic way?  "jfr-8u" branch in a jdk8u-dev/sandbox forest, perhaps?
>
> My understanding was that this was dismissed, but I'm happy either way.
>
> If we have an agreement on how to call the repository then Andrew will
> create one right away.
>
> Btw, I think the discussion should move to jdk8u-dev alone. I'm not
> sure if we will want to coordinate this effort on a separate mailing
> list though, what do you think?

I think it would be cleaner to have multiple repositories: having to deal
with branches make things pointlessly difficult. Maybe we should give this
some structure with subdirectories, so jdk8u/incubator/jfr. The advantage
of this is that a casual visitor is less likely to be confused.

I like this proposal, it is future proof as it would make it clear also if we ever had to add more of such repositories what they are for.

Cheers,
Mario

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



More information about the jdk8u-dev mailing list