> 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. ------------- Changes: - all: https://git.openjdk.org/jdk/pull/21040/files - new: https://git.openjdk.org/jdk/pull/21040/files/aa22301c..fabcaebe Webrevs: - full: https://webrevs.openjdk.org/?repo=jdk&pr=21040&range=01 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=21040&range=00-01 Stats: 198321 lines in 1421 files changed: 184773 ins; 6839 del; 6709 mod Patch: https://git.openjdk.org/jdk/pull/21040.diff Fetch: git fetch https://git.openjdk.org/jdk.git pull/21040/head:pull/21040 PR: https://git.openjdk.org/jdk/pull/21040