On Wed, 19 May 2021 10:00:04 GMT, Severin Gehwolf <[email protected]> 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
