RFR: 7167: Agent Plugin [v11]

Brice Dutheil github.com+803621+bric3 at openjdk.java.net
Fri Jun 25 21:34:06 UTC 2021


On Tue, 22 Jun 2021 22:10:08 GMT, Joshua Matsuoka <jmatsuoka at openjdk.org> wrote:

>> This PR adds the Agent Plugin to JMC. This is a plugin for controlling the jmc agent and provides a way to attach it and add probes to any JVMs in the JVM browser, as well as providing a preset manager for creating/modifying probe presets.
>> 
>> There is still a small amount of cleanup to be done so I'll mark it as a draft for now but it would be great to get a review :)
>
> Joshua Matsuoka has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Decorating outputstreamWriter with a bufferedWriter

Yes it looks good to me.

application/org.openjdk.jmc.console.agent/src/main/java/org/openjdk/jmc/console/agent/editor/AgentEditorUi.java line 191:

> 189: 				DialogToolkit.showWarning(window.getShell(), Messages.AgentEditorUI_MESSAGE_EMPTY_PRESET_TITLE,
> 190: 						Messages.AgentEditorUI_MESSAGE_EMPTY_PRESET);
> 191: 			}

��

application/org.openjdk.jmc.console.agent/src/main/java/org/openjdk/jmc/console/agent/wizards/StartAgentWizard.java line 66:

> 64: 		startAgentWizardPage = new StartAgentWizardPage(helper);
> 65: 		this.setHelpAvailable(false);
> 66: 		WizardDialog.setDialogHelpAvailable(false);

��

-------------

Marked as reviewed by bric3 at github.com (no known OpenJDK username).

PR: https://git.openjdk.java.net/jmc/pull/226


More information about the jmc-dev mailing list