RFR: 8338603: DiagnosticCommandMBean operations should standardize types for parameters [v2]

Erik Gahlin egahlin at openjdk.org
Tue Oct 15 08:34:13 UTC 2024


On Thu, 10 Oct 2024 08:51:38 GMT, Kevin Walls <kevinw at openjdk.org> wrote:

>> DiagnosticCommandImpl should only publish parameter types in a known standard set, and use "STRING" on anything else.
>> e.g. We can say "FILE" in the help output for jcmd, as that's for humans, but the MBean parameter info should contain "STRING".
>
> Kevin Walls 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 four additional commits since the last revision:
> 
>  - Merge remote-tracking branch 'upstream/master' into 8338603_dcmd_mbean_types
>  - Update comment about type name knowledge
>  - Test update
>  - 8338603: DiagnosticCommandMBean operations should standardize types for parameters.

Marked as reviewed by egahlin (Reviewer).

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

PR Review: https://git.openjdk.org/jdk/pull/21040#pullrequestreview-2368548822


More information about the serviceability-dev mailing list