<html>
  <head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <br>
    <br>
    <div class="moz-cite-prefix">On 10/25/18 2:52 AM, Daniel Fuchs
      wrote:<br>
    </div>
    <blockquote type="cite"
      cite="mid:b04dbd64-4173-8ddd-349c-e06b176ced70@oracle.com">Hi
      Mandy,
      <br>
      <br>
      I agree that this looks more robust and will be better for
      <br>
      long term maintainability. I'm just surprised that
      <br>
      <br>
       156     static CompositeType compositeType() {
      <br>
       157         return STACK_TRACE_ELEMENT_COMPOSITE_TYPE;
      <br>
       158     }
      <br>
      <br>
      is no longer (or was never) needed in
      StackTraceElementCompositeData
      <br>
      when
      <br>
      <br>
       146     static CompositeType v5CompositeType() {
      <br>
       147         return V5_COMPOSITE_TYPE;
      <br>
       148     }
      <br>
      <br>
      appears to still be needed.
      <br>
      <br>
    </blockquote>
    <br>
    It's used by MonitorInfoCompositeInfo and ThreadInfoCompositeInfo to
    build their CompositeType of older version.  For the current
    version, it gets it from MappedMXBeanType.toOpenType and hence no
    need for compositeType().<br>
    <br>
    <blockquote type="cite"
      cite="mid:b04dbd64-4173-8ddd-349c-e06b176ced70@oracle.com">Otherwise,
      this looks good to me.
      <br>
    </blockquote>
    <br>
    Thanks for the review.<br>
    <br>
    Mandy<br>
    <br>
    <blockquote type="cite"
      cite="mid:b04dbd64-4173-8ddd-349c-e06b176ced70@oracle.com">
      <br>
      best regards,
      <br>
      <br>
      -- daniel
      <br>
      <br>
      On 24/10/2018 23:53, Mandy Chung wrote:
      <br>
      <blockquote type="cite">This patch fixes the regression introduced
        by JDK-8198253 in 11.
        <br>
        It turns out that NetBeans uses the internal sun.management API
        to
        <br>
        convert ThreadInfo to CompositeData for performance reason.
        <br>
        ThreadInfoCompositeData::toCompositeData is no longer used
        <br>
        in JDK since JMX added the MXBean support in JDK 6. The fix for
        <br>
        JDK-8212197 resolves one issue reported [1] but not the bug in
        <br>
        ThreadInfoCompositeData::toCompositeData. Sven has filed an
        <br>
        issue in NetBeans to replace the use of JDK internal API.
        <br>
        <br>
        Webrev:
        <br>
<a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~mchung/jdk12/webrevs/8212795/webrev.00/">http://cr.openjdk.java.net/~mchung/jdk12/webrevs/8212795/webrev.00/</a>
        <br>
        <br>
        Thanks
        <br>
        Mandy
        <br>
        [1]
<a class="moz-txt-link-freetext" href="http://mail.openjdk.java.net/pipermail/serviceability-dev/2018-October/025512.html">http://mail.openjdk.java.net/pipermail/serviceability-dev/2018-October/025512.html</a><br>
        [2] <a class="moz-txt-link-freetext" href="https://issues.apache.org/jira/browse/NETBEANS-1478">https://issues.apache.org/jira/browse/NETBEANS-1478</a>
        <br>
      </blockquote>
      <br>
    </blockquote>
    <br>
  </body>
</html>