RFR: 8203359: Container level resources events [v11]
Jaroslav Bachorik
jbachorik at openjdk.java.net
Wed May 19 10:27:42 UTC 2021
On Wed, 19 May 2021 10:00:04 GMT, Severin Gehwolf <sgehwolf at openjdk.org> wrote:
>> Jaroslav Bachorik has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains 10 commits:
>>
>> - Small fixes
>> - Remove trailing spaces
>> - Doh
>> - Report container type and register events conditionally
>> - Remove unused test files
>> - Initial test support for JFR container events
>> - Update the JFR control files
>> - Split off the CPU throttling metrics
>> - Formatting spaces
>> - 8203359: Container level resources events
>
> src/jdk.jfr/share/classes/jdk/jfr/events/ContainerMemoryUsageEvent.java line 48:
>
>> 46: @Description("(attempts per second * 1000), if enabled, that the operating system tries to satisfy a memory request for any " +
>> 47: "process in the current container when no free memory is readily available.")
>> 48: public double memoryPressure;
>
> Should this `memoryPressure` field go from `ContainerMemoryUsageEvent` class? It's not set anywhere is it? would be cgroup v1 only api so I'm not sure it should be there for a generic event like this.
Yes. Removing.
-------------
PR: https://git.openjdk.java.net/jdk/pull/3126
More information about the core-libs-dev
mailing list