RFR: 8288783: JFR: Error messages are confusing when options conflict in -XX:StartFlightRecording [v2]
Chihiro Ito
cito at openjdk.org
Thu Oct 13 03:20:17 UTC 2022
> Could I have a review of PR that fixes incorrect error messages regarding of recording.
>
> The current message is very confusing. If a user types jcmd <pid> JFR.start name=abc name=def or java --XX:StartFlightRecording=name=abc,name=def, the error message says "Duplicates in diagnostic command arguments", but doesn't know what options are duplicated.
> Furthermore, if a user specifies two or three duplicate parameters, the same logs will be output.
>
> It should say follows,
> Option name can only be specified once with starting flight recording
> Options name and disk can only be specified once with starting flight recording
> Options name, disk and maxage can only be specified once with starting flight recording
> This problem affects --XX:StartFlightRecording, jcmd JFR.start, JFR.stop, JFR.dump, JFR.check, and MBean.
>
> For other than start, the output is as follows, (example of dump)
> Option name can only be specified once with dumping flight recording
>
> Testing: jdk/jdk/jfr
>
> Thanks
> Chihiro
Chihiro Ito has updated the pull request incrementally with one additional commit since the last revision:
eliminating complication such as outputting commands, and changing how to build log messages
-------------
Changes:
- all: https://git.openjdk.org/jdk/pull/9302/files
- new: https://git.openjdk.org/jdk/pull/9302/files/1489318a..52316757
Webrevs:
- full: https://webrevs.openjdk.org/?repo=jdk&pr=9302&range=01
- incr: https://webrevs.openjdk.org/?repo=jdk&pr=9302&range=00-01
Stats: 158 lines in 10 files changed: 3 ins; 105 del; 50 mod
Patch: https://git.openjdk.org/jdk/pull/9302.diff
Fetch: git fetch https://git.openjdk.org/jdk pull/9302/head:pull/9302
PR: https://git.openjdk.org/jdk/pull/9302
More information about the hotspot-jfr-dev
mailing list