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

Remove dead code and clean up jvmstat LocalVmManager

    XMLWordPrintable

    Details

    • Type: Enhancement
    • Status: Resolved
    • Priority: P4
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 18
    • Component/s: core-svc
    • Labels:
    • Subcomponent:
    • Resolved In Build:
      b23

      Description

      In the jdk.internal.jvmstat module, the LocalVmManager(String user) constructor is supposed for be used to look for VMs owned by the user. See

      https://github.com/openjdk/jdk/blob/b1b83500a9c3a74bf39894e49eefd031d208b9b9/src/jdk.internal.jvmstat/share/classes/sun/jvmstat/perfdata/monitor/protocol/local/LocalVmManager.java#L77

      However, no code calls this constructor. Also, even if LocalVmManager.userName is set, the current implementation doesn't look for VMs owned by this user.

      This constructor should be removed, as well as all code that depends on LocalVmManager.userName.

      There's related unused code in the PerfDataFile class in the same package.

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              iklam Ioi Lam
              Reporter:
              iklam Ioi Lam
              Votes:
              0 Vote for this issue
              Watchers:
              3 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved: