jmx-dev RFR: 8335625: Update Javadoc for GetCpuLoad [v3]

[Joakim Nordström]

On Fri, 23 Aug 2024 12:01:43 GMT, Alan Bateman <alanb at openjdk.org> wrote:

>> Joakim Nordström has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Added implNote and made some re-arrangements
>
> src/jdk.management/share/classes/com/sun/management/OperatingSystemMXBean.java line 154:
> 
>> 152:      * affecting the observation period and the result.
>> 153:      *
>> 154:      * @implNote There is only one observation period for the entire JVM
> 
> This version looks good, I'm just wondering about the implNote, is the statement true for all implementations (Linux, macOS, Windows, ...)?

Reviewing the current implementation this seems to be the case. 

The `@implNote` was more meant as an explanation to why the result could be affected by other callers. Since the `@apiNote` also states that the "observation period and result" can be affected, I think its an unnecessary implementation note to have.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/20546#discussion_r1731275288