Integrated: 8335625: Update Javadoc for GetCpuLoad

2024-09-10 Thread Joakim Nordström
On Mon, 12 Aug 2024 12:33:04 GMT, Joakim Nordström wrote: > Can I get a review of this documentation update to clarify the usage of > GetCpuLoad (and inherently deprecated GetSystemCpuLoad) and GetProcessCpuLoad. > > Calling either of these methods in quick succession

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v4]

2024-09-10 Thread Joakim Nordström
On Mon, 26 Aug 2024 13:54:36 GMT, Joakim Nordström wrote: >> Can I get a review of this documentation update to clarify the usage of >> GetCpuLoad (and inherently deprecated GetSystemCpuLoad) and >> GetProcessCpuLoad. >> >> Calling either of these methods i

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v3]

2024-08-26 Thread Joakim Nordström
On Mon, 26 Aug 2024 13:51:08 GMT, Alan Bateman wrote: >> 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 pe

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v3]

2024-08-26 Thread Joakim Nordström
On Fri, 23 Aug 2024 12:01:43 GMT, Alan Bateman 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/sh

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v4]

2024-08-26 Thread Joakim Nordström
entative values, since getSystemCpuLoad effectively calls > getCpuLoad. Joakim Nordström has updated the pull request incrementally with one additional commit since the last revision: Removed implNote - Changes: - all: https://git.openjdk.org/jdk/pull/20546/files - new: htt

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v3]

2024-08-23 Thread Joakim Nordström
On Fri, 23 Aug 2024 08:49:38 GMT, Joakim Nordström wrote: >> Can I get a review of this documentation update to clarify the usage of >> GetCpuLoad (and inherently deprecated GetSystemCpuLoad) and >> GetProcessCpuLoad. >> >> Calling either of these methods i

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v3]

2024-08-23 Thread Joakim Nordström
entative values, since getSystemCpuLoad effectively calls > getCpuLoad. Joakim Nordström has updated the pull request incrementally with one additional commit since the last revision: Added implNote and made some re-arrangements - Changes: - all: https://git.openjdk.org/jd

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v2]

2024-08-21 Thread Joakim Nordström
On Wed, 21 Aug 2024 22:27:27 GMT, David Holmes wrote: >> I want to provide advice for how this API should be used. >> >> The "recent period of time observed" is vague (Who's the observer? Who >> decides the "recent period"? The API user or the JVM?). In essence it is the >> time between two co

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v2]

2024-08-21 Thread Joakim Nordström
On Wed, 21 Aug 2024 12:25:48 GMT, Kevin Walls wrote: >> You beat me to it. We specify some methods (close, dispose, ...) as >> idempotent. Specifying a getter as non-idempotent looks very strange here. >> >> I can't tell if you are looking to specify implementation behavior (as David >> asks)

Re: RFR: 8335625: Update Javadoc for GetCpuLoad [v2]

2024-08-21 Thread Joakim Nordström
entative values, since getSystemCpuLoad effectively calls > getCpuLoad. Joakim Nordström has updated the pull request incrementally with one additional commit since the last revision: Made it an apiNote - Changes: - all: https://git.openjdk.org/jdk/pull/20546/files - new: htt

RFR: 8335625: Update Javadoc for GetCpuLoad

2024-08-21 Thread Joakim Nordström
Can I get a review of this documentation update to clarify the usage of GetCpuLoad (and inherently deprecated GetSystemCpuLoad) and GetProcessCpuLoad. Calling either of these methods in quick succession can lead to unrepresentative results due to too few data points. This behavior is easy to re