RFR: 8257424: Add setName() to RecordingStream

Erik Gahlin egahlin at openjdk.java.net
Mon Nov 30 23:13:03 UTC 2020


On Mon, 30 Nov 2020 15:00:39 GMT, Yasumasa Suenaga <ysuenaga at openjdk.org> wrote:

> `RecordingStream` will start new flight recording to gather events. It has some of methods which are equivalent to `Recording`, but we cannot set recording name.
> 
> If we can specify the name for `RecordingStream`, it is useful to distinguish recordings by JFR.check dcmd.

The underlying recording is an implementation detail, not sure it should show up in jcmd at all. Also, RemoteRecordingStream and RecordingStream should be interchangeable and setting the name on RemoteRecordingStream will not work with the current implementation. 

We have deliberately not added methods in Recording to RecordingStream and kept the API to the bare minimum Many of the methods are trivial to implement, but we don't want to commit on them yet. Instead we like to get the overall architecture correct and there still things we need to figure out.

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

PR: https://git.openjdk.java.net/jdk/pull/1520


More information about the hotspot-jfr-dev mailing list