On Thu, 10 Oct 2024 08:51:38 GMT, Kevin Walls <kev...@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 cjplummer (Reviewer). ------------- PR Review: https://git.openjdk.org/jdk/pull/21040#pullrequestreview-2361929093
