[11u] RFR(M): 8261447 backport: MethodInvocationCounters frequently run into overflow

[Schmidt, Lutz]

Hey Paul,
you are my hero!
Thanks and have a great weekend,
Lutz

On 16.04.21, 17:38, "Hohensee, Paul" <hohensee at amazon.com> wrote:

    I'll sponsor.

    -----Original Message-----
    From: "Schmidt, Lutz" <lutz.schmidt at sap.com>
    Date: Friday, April 16, 2021 at 6:10 AM
    To: "Hohensee, Paul" <hohensee at amazon.com>, "jdk-updates-dev at openjdk.java.net" <jdk-updates-dev at openjdk.java.net>
    Cc: "serviceability-dev at openjdk.java.net" <serviceability-dev at openjdk.java.net>
    Subject: RE: [11u] RFR(M): 8261447 backport: MethodInvocationCounters frequently run into overflow

    Hi Paul,

    thank you for the review.

    @All: It's annoying, I know. I need somebody to sponsor this change as well.

    Thanks, Lutz


    On 15.04.21, 22:45, "Hohensee, Paul" <hohensee at amazon.com> wrote:

        Lgtm.

        Thanks,
        Paul

        -----Original Message-----
        From: serviceability-dev <serviceability-dev-retn at openjdk.java.net> on behalf of "Schmidt, Lutz" <lutz.schmidt at sap.com>
        Date: Thursday, April 15, 2021 at 1:19 AM
        To: "jdk-updates-dev at openjdk.java.net" <jdk-updates-dev at openjdk.java.net>
        Cc: "serviceability-dev at openjdk.java.net" <serviceability-dev at openjdk.java.net>
        Subject: [11u] RFR(M): 8261447 backport: MethodInvocationCounters frequently run into overflow

        Dear Community,

        I would happily accept reviews for this downport change. The change creates more meaningful output.

        Merge conflicts were only "technical". No complicated semantical considerations necessary.

        Original bug:          https://bugs.openjdk.java.net/browse/JDK-8261447
        Downport webrev:       https://cr.openjdk.java.net/~lucy/webrevs/8261447.11u.01/

        Tests:
        SAP's internal build and test farm (all OpenJDK platforms (no 32-bit), and more). Tests include JCK, jtreg (hotspot and jdk), and SAP-private tests. No issues detected that are attributable to this change.

        Thanks,
        Lutz