RFR: Backport of JDK-8044135: Add API to start JMX agent from attach framework

Dmitry Samersoff dmitry.samersoff at oracle.com
Thu Sep 4 11:52:17 UTC 2014


Staffan,

WindowsVirtualMachine.java:109 Should we close PipedInputStream ?

Otherwise looks good.

-Dmitry

On 2014-09-04 14:11, Staffan Larsen wrote:
> Can I interest any Reviewers in taking a look at this?
> 
> Thanks,
> /Staffan
> 
> On 3 sep 2014, at 14:27, Staffan Larsen <staffan.larsen at oracle.com> wrote:
> 
>> …aaaand the link: http://cr.openjdk.java.net/~sla/8044398-8039173-8044135-jdk8u/webrev.00/
>>
>> /Staffan
>>
>> On 3 sep 2014, at 14:25, Staffan Larsen <staffan.larsen at oracle.com> wrote:
>>
>>> This is a review for a backport of JDK-8044135 to jdk8u. This fix had a few dependencies that also needed backporting, so included in this review are the following fixes from JDK9:
>>>
>>> • 8044135 Add API to start JMX agent from attach framework (jdk)
>>> • 8039173 Propagate errors from Diagnostic Commands as exceptions in the attach framework (jdk)
>>> • 8044398 Attach code should propagate errors in Diagnostic Commands as errors  (hotspot)
>>>
>>> The only significant change from the version in JDK 9 is that startManagementAgent() and startLocalManagementAgent() in VirtualMachine.java have "@since 1.8" (instead of 1.9).
>>>
>>> After speaking with the maintainers and gatekeepers, I plan to push this to the jdk8u-hs-dev repository.
>>>
>>> Thanks,
>>> /Staffan
>>>
>>
> 


-- 
Dmitry Samersoff
Oracle Java development team, Saint Petersburg, Russia
* I would love to change the world, but they won't give me the sources.


More information about the serviceability-dev mailing list