Uploaded image for project: 'JDK'
  1. JDK
  2. JDK-8288741

JFR: Change package name of snippet files

    XMLWordPrintable

Details

    • Bug
    • Status: Resolved
    • P3
    • Resolution: Fixed
    • 19
    • 19
    • hotspot
    • None
    • jfr
    • b28

    Backports

      Description

        Not sure if it is by design, but when I look at jrt-fs.jar, I can see snippet classes being include. In the JFR case, the packages are "example1" and "example2".

        This looks strange and adds noise when looking at JFR classes in an IDE (Eclipse). Better package names would be "jdk.jfr.snippets" and "jdk.jfr.snippets.consumer"

        Attachments

          Issue Links

            backported by

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8288778 JFR: Change package name of snippet files

            • P3 - Major loss of function.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8291104 JFR: Change package name of snippet files

            • P3 - Major loss of function.
            • Resolved

            Backport - A issue that is required to port a Bug or Feature into another product release. This issue type is generally associated with the main Bug/Feature to represent each individual release of the port. JDK-8291271 JFR: Change package name of snippet files

            • P3 - Major loss of function.
            • Resolved
            links to

            Commit Commit openjdk/jdk19/33d0363e

            Review Review openjdk/jdk19/46

            Activity

              People

                egahlin Erik Gahlin
                egahlin Erik Gahlin
                Votes:
                0 Vote for this issue
                Watchers:
                3 Start watching this issue

                Dates

                  Created:
                  Updated:
                  Resolved: