
<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

Hi Carter,</div>

<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

<br>

</div>

<div class="elementToProof" style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">

I have filed a bug report. I will look into your fix tomorrow. </div>

<div class="elementToProof"><span style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"><a href="https://bugs.openjdk.org/browse/JDK-8322142">https://bugs.openjdk.org/browse/JDK-8322142</a></span></div>

<div class="elementToProof"><span style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);"><br>

</span></div>

<div class="elementToProof"><span style="font-family: Aptos, Aptos_EmbeddedFont, Aptos_MSFontService, Calibri, Helvetica, sans-serif; font-size: 12pt; color: rgb(0, 0, 0);">Erik</span></div>

<div id="appendonsend"></div>

<hr style="display:inline-block;width:98%" tabindex="-1">

<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Carter Kozak <ckozak@ckozak.net><br>

<b>Sent:</b> Friday, December 15, 2023 12:13 AM<br>

<b>To:</b> Erik Gahlin <erik.gahlin@oracle.com>; hotspot-jfr-dev@openjdk.java.net <hotspot-jfr-dev@openjdk.java.net><br>

<b>Subject:</b> [External] : Re: JFR Recordings Missing Periodic Events</font>

<div> </div>

</div>

<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">

<div class="PlainText">Hi Erik,<br>

<br>

Thanks for the quick reply!<br>

<br>

I've simplified my reproducer based on your recommendations, however the result is the same. To clarify the issue I'm facing, I'm not worried about the EventStream/RecordingStream so much as the resulting JFR itself, which is missing periodic event data.<br>

The CI output includes the updated logging, for example:<br>

<a href="https://urldefense.com/v3/__https://github.com/carterkozak/periodic-event-repro/actions/runs/7215619705/job/19660234573__;!!ACWV5N9M2RV99hQ!OAej4SiBWTTz-IgdpnU3OtixXqEWHZoghi9QqV7eOVcz-5HOalkFt7SiI_beWUIMrSIYZgoxcDuen_1bww$">https://urldefense.com/v3/__https://github.com/carterkozak/periodic-event-repro/actions/runs/7215619705/job/19660234573__;!!ACWV5N9M2RV99hQ!OAej4SiBWTTz-IgdpnU3OtixXqEWHZoghi9QqV7eOVcz-5HOalkFt7SiI_beWUIMrSIYZgoxcDuen_1bww$</a>

<br>

<br>

I've put together a quick draft based on the proposal in my last reply, I have confirmed that it resolves the failures:<br>

<a href="https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/17114__;!!ACWV5N9M2RV99hQ!OAej4SiBWTTz-IgdpnU3OtixXqEWHZoghi9QqV7eOVcz-5HOalkFt7SiI_beWUIMrSIYZgoxcDvdxiNkkg$">https://urldefense.com/v3/__https://github.com/openjdk/jdk/pull/17114__;!!ACWV5N9M2RV99hQ!OAej4SiBWTTz-IgdpnU3OtixXqEWHZoghi9QqV7eOVcz-5HOalkFt7SiI_beWUIMrSIYZgoxcDvdxiNkkg$</a>

<br>

<br>

Carter Kozak<br>

<br>

On Thu, Dec 14, 2023, at 17:12, Erik Gahlin wrote:<br>

> Hi Carter,<br>

> <br>

> t's not possible to guarantee that events will be available unless a RecordingStream is used, the period is everyChunk and the stop() method is used.<br>

> <br>

> That said, it looks strange. You could try running with -Xlog:jfr+system+parser=info,jfr+periodic=debug and only enable the ThreadStatistics event:<br>

> <br>

> recording.enable("jdk.JavaThreadStatistics").with("period", "1000 ms"); <br>

> <br>

> It might make it easier to diagnose.<br>

> <br>

> Erik<br>

</div>

</span></font></div>

</body>

</html>