RFR: 8284686: Interval of < 1 ms disables ExecutionSample events [v6]

[Christoph Langer]

On Thu, 5 May 2022 11:02:09 GMT, Johannes Bechberger <duke at openjdk.java.net> wrote:

>> Fixes the issue by rounding the period interval to the next full millisecond larger than zero.
>> 
>> The following from the bug report works now:
>> 
>> java -XX:StartFlightRecording=filename=flight.jfr,jdk.ExecutionSample#period=999us CLASS_FILE; jfr print --events jdk.ExecutionSample flight.jfr
>> 
>> It considers the period to be `1ms`, producing the recording as expected.
>
> Johannes Bechberger has updated the pull request incrementally with two additional commits since the last revision:
> 
>  - Add TestShortPeriod
>  - Remove TestPeriodSetting

test/jdk/jdk/jfr/api/recording/event/TestShortPeriod.java line 2:

> 1: /*
> 2:  * Copyright (c) 2022, Oracle and/or its affiliates. All rights reserved.

@parttimenerd Since you wrote the test, it should have SAP copyright.

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

PR: https://git.openjdk.java.net/jdk/pull/8183