RFR(XL): [JEP-349] 8226511: Implement JFR Event Streaming

mikhailo.seledtsov at oracle.com mikhailo.seledtsov at oracle.com
Thu Sep 19 18:00:29 UTC 2019


Hi Markus,

    I have reviewed tests only. Overall looks good.

I have a number of small corrections:

     - there are several wrong targets for @run
     - I propose to add logs to beginnings and ends of test cases that 
might time out; this way we will know which test case within a test 
file/class timed out
     - I have found several typos

Here is the proposed patch that contains these fixes and corrections:

http://cr.openjdk.java.net/~mseledtsov/8226511.test.01/index.html


Thank you,

Misha


On 9/18/19 12:48 PM, Markus Gronlund wrote:
> Greetings,
>
> This is a RFR for JEP 349: JFR Event Streaming [1].
>
> The integration work is tracked under JBS issue 8226511 [2] with associated webrev [3], CSR [4] and JavaDocs [5].
>
> The code includes a demo example [6], provided for convenience to get a taste of what can be built on top of this feature*.
> The demo can be activated by starting a regular recording using a specific name, "health", i.e. -XX:StartFlightRecording:name=health
>
> Thank you
> Markus and Erik
>
> [1] http://openjdk.java.net/jeps/349
> [2] https://bugs.openjdk.java.net/browse/JDK-8226511
> [3] http://cr.openjdk.java.net/~mgronlun/8226511/v0/webrev/webrev/
> [4] https://bugs.openjdk.java.net/browse/JDK-8226512
> [5] http://cr.openjdk.java.net/~mgronlun/8226511/v0/docs/docs/api/jdk.jfr/jdk/jfr/consumer/package-summary.html
> [6] http://cr.openjdk.java.net/~egahlin/health-report-agent/health-report-example.txt
> *   The demo example code is only included for convenience, it will be removed later.
>


More information about the hotspot-jfr-dev mailing list