RFR: 8298175: JFR: Common timestamp for periodic events [v2]

Markus Grönlund mgronlun at openjdk.org
Wed Dec 7 14:23:42 UTC 2022


On Wed, 7 Dec 2022 07:07:13 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

>> Could I have review of infrastructure that makes it easier to write periodic events with the same timestamp.
>> 
>> Testing: jdk/jdk/jfr + tier1 + tier2
>> 
>> Thanks
>> Erik
>
> Erik Gahlin has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Update request loop

Marked as reviewed by mgronlun (Reviewer).

src/jdk.jfr/share/classes/jdk/jfr/internal/RequestEngine.java line 210:

> 208:     private static long run_requests(Collection<RequestHook> entries, long eventTimestamp) {
> 209:         long last = lastTimeMillis;
> 210:         // The interval for periodic events are typically at least 1 s, so

"are" -> "is"

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

PR: https://git.openjdk.org/jdk/pull/11541



More information about the build-dev mailing list