RFR: 8286396: Address possibly lossy conversions in jdk.management.jfr [v2]

Markus Grönlund mgronlun at openjdk.java.net
Wed May 11 12:00:59 UTC 2022


On Wed, 11 May 2022 11:29:29 GMT, Erik Gahlin <egahlin at openjdk.org> wrote:

>> Could I have a review of two lossy conversion issues.
>> 
>> The size of an event can't be larger than 2^28, so it shouldn't be an issue in practise, but the type id could potentially be larger than 2^31, but it would require a user to load more than 2 billion classes. 
>> 
>> Testing: jdk/jdk/jfr
>> 
>> Thanks
>> Erik
>
> Erik Gahlin has updated the pull request with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains two additional commits since the last revision:
> 
>  - Merge branch 'openjdk:master' into lossy_jdkmanagementjfr
>  - Initial

Marked as reviewed by mgronlun (Reviewer).

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

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


More information about the hotspot-jfr-dev mailing list