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