Review Request JDK-8212795: ThreadInfoCompositeData.toCompositeData fails to map ThreadInfo to CompositeData

Daniel Fuchs daniel.fuchs at
Thu Oct 25 09:52:23 UTC 2018

Hi Mandy,

I agree that this looks more robust and will be better for
long term maintainability. I'm just surprised that

  156     static CompositeType compositeType() {
  158     }

is no longer (or was never) needed in StackTraceElementCompositeData

  146     static CompositeType v5CompositeType() {
  147         return V5_COMPOSITE_TYPE;
  148     }

appears to still be needed.

Otherwise, this looks good to me.

best regards,

-- daniel

On 24/10/2018 23:53, Mandy Chung wrote:
> This patch fixes the regression introduced by JDK-8198253 in 11.
> It turns out that NetBeans uses the internal API to
> convert ThreadInfo to CompositeData for performance reason.
> ThreadInfoCompositeData::toCompositeData is no longer used
> in JDK since JMX added the MXBean support in JDK 6. The fix for
> JDK-8212197 resolves one issue reported [1] but not the bug in
> ThreadInfoCompositeData::toCompositeData. Sven has filed an
> issue in NetBeans to replace the use of JDK internal API.
> Webrev:
> Thanks
> Mandy
> [1] 
> [2]

More information about the serviceability-dev mailing list