RFR: 8254001: [Metrics] Enhance parsing of cgroup interface files for version detection [v4]

Severin Gehwolf sgehwolf at openjdk.java.net
Fri Feb 12 10:21:48 UTC 2021


On Thu, 11 Feb 2021 19:51:43 GMT, Harold Seigel <hseigel at openjdk.org> wrote:

>> Severin Gehwolf 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 seven additional commits since the last revision:
>> 
>>  - Fix jcheck
>>  - Add documentation and reduce code running in the critical section
>>  - Add some documentation
>>  - Merge branch 'master' into jdk-8254001-enhance-file-parsing-java-metrics
>>  - Merge branch 'master' into jdk-8254001-enhance-file-parsing-java-metrics
>>  - Merge branch 'master' into jdk-8254001-enhance-file-parsing-java-metrics
>>  - 8254001: [Metrics] Enhance parsing of cgroup interface files for version detection
>
> Hi Severin,
> Thanks for doing this!  Sorry for taking so long to review this change.  The change looks good.  Before pushing it, could you add a comment explaining what the code in lines 185-194 of CgroupSubsystemFactory.java is doing?  Also, please don't overwrite the fix for JDK-8257746.
> Thanks again! Harold

Setting reviewers to 2 since I want @gnu-andrew to be OK with it too.

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

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


More information about the core-libs-dev mailing list