Skip to content

Conversation

@kisstibor
Copy link

Extend the JMXQuery library to let the user invoke JMX methods. This would be very useful to create custom reports, where we can trigger actions as well.

@kisstibor kisstibor force-pushed the jmx_method_invokation branch from 2d427bb to fb129f7 Compare July 9, 2019 13:43
This would be very useful to create custom reports, where we can trigger
actions as well.
@kisstibor kisstibor force-pushed the jmx_method_invokation branch from fb129f7 to 8fcd43f Compare July 9, 2019 14:00
this method return the json serialization of this object
:return: json string
"""
json_param = ",".join(["{\"value\":\""+param.value+"\", type:\""+param.value_type+"\"}" for param in self.params])
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This JSON is incorrect that occurred an error in Java.

Suggested change
json_param = ",".join(["{\"value\":\""+param.value+"\", type:\""+param.value_type+"\"}" for param in self.params])
json_param = ",".join(["{\"value\":\""+param.value+"\"," type":\""+param.value_type+"\"}" for param in self.params])

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants