You are viewing a plain text version of this content. The canonical link for it is here.
Posted to jmeter-dev@jakarta.apache.org by Kumara Subramanya <ku...@gmail.com> on 2006/06/10 16:39:10 UTC
The jmx sampler patch.
Hi,
I wrote a plugin to enable sampling of mbean servers in jmeter. (refer
to bug 39642).
I added the source as an attachment to the issue in bugzilla.
Regards
Subrahmanya A
--
--
Subrahmanya Adiyanadka
--
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
Re: The jmx sampler patch.
Posted by sebb <se...@gmail.com>.
On 11/06/06, Kumara Subramanya <ku...@gmail.com> wrote:
> The stuff has no external dependecies.
What about the JMX libraries? These are not in my JDK 1.4.
> The patch should perfectly work under jdk1.4 as per my knowledge.
I found a call to new PrintWriter(String) which is @since 1.5
Easy to fix, but I don't believe the code should be writing files
anyway, that should be left to the existing code.
I think the JMXSampleResult class should be removed; it's just going
to complicate the JTL files.
I don't understand how the visualiser is supposed to work. Does it do
anything other than write to the result file? If not, why do you need
a different visualiser?
To allow use in non-GUI tests, it's important that the GUI elements
only do processing which is related to the GUI.
Have a look a AssertionVisualizer for an example of a simple visualiser.
All it does is implement the add(SampleResult) method. All the code to
write samples to the JTL files is handled by existing JMeter classes.
The visualiser only updates the GUI; it is not involved at all in
non-GUI tests.
> I forgot to delete the dist and syn directories when i created the
> jar. Sorry for that.
There were no such directories in the zip/jar file, only
plugin
plugin/jmx etc
> As per licence is concerned would it be cool if I just copy paste the
> apache licence statement into each source files ?.
I need to check that. We may need a CLA as well, as there is a fair
amount of code involved.
> Let me know.
I think it will be useful code, but the visualiser area needs a bit of
work to fit in better with the existing JMeter design.
Also, there seems to be a problem with the Sampler GUI. When clicking
on the "Add MBean" button, I get the following message in the console:
java.lang.Error: Do not use org.apache.jmeter.protocol.jmx.control.gui.MBeanTree
.add() use org.apache.jmeter.protocol.jmx.control.gui.MBeanTree.getContentPane()
.add() instead
at javax.swing.JDialog.createRootPaneException(JDialog.java:536)
at javax.swing.JDialog.addImpl(JDialog.java:562)
at java.awt.Container.add(Container.java:307)
at org.apache.jmeter.protocol.jmx.control.gui.MBeanTree.initgui(MBeanTre
e.java:89)
[...]
Try running the code in non-GUI and client-server mode. This is a good
way of finding out if the code is in the right place...
S
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
Re: The jmx sampler patch.
Posted by Kumara Subramanya <ku...@gmail.com>.
The stuff has no external dependecies.
The patch should perfectly work under jdk1.4 as per my knowledge.
I forgot to delete the dist and syn directories when i created the
jar. Sorry for that.
As per licence is concerned would it be cool if I just copy paste the
apache licence statement into each source files ?.
Let me know.
Regards
Subrahmanya A
On 6/10/06, sebb <se...@gmail.com> wrote:
> It would be helpful if you could provide a list of external
> dependencies, and where to obtain them - but please don't upload the
> jars to Bugzilla.
>
> Also does the plugin work with Java 1.4, or does it require 1.5?
>
> S.
> On 10/06/06, sebb <se...@gmail.com> wrote:
> > Thanks, I saw that.
> >
> > Unfortunately it's too late for the release I'm currently working on.
> >
> > However, it is self-contained, so it you attached the jar as well,
> > users could try it if they wanted.
> >
> > Please remember to set the correct mime-type for any attachments.
> >
> > Also, it needs a license.
> >
> > S.
> > On 10/06/06, Kumara Subramanya <ku...@gmail.com> wrote:
> > > Hi,
> > >
> > > I wrote a plugin to enable sampling of mbean servers in jmeter. (refer
> > > to bug 39642).
> > > I added the source as an attachment to the issue in bugzilla.
> > >
> > >
> > > Regards
> > > Subrahmanya A
> > > --
> > > --
> > > Subrahmanya Adiyanadka
> > > --
> > >
> > > ---------------------------------------------------------------------
> > > To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
> > > For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
> > >
> > >
> >
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
>
>
--
--
Subrahmanya Adiyanadka
--
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
Re: The jmx sampler patch.
Posted by sebb <se...@gmail.com>.
It would be helpful if you could provide a list of external
dependencies, and where to obtain them - but please don't upload the
jars to Bugzilla.
Also does the plugin work with Java 1.4, or does it require 1.5?
S.
On 10/06/06, sebb <se...@gmail.com> wrote:
> Thanks, I saw that.
>
> Unfortunately it's too late for the release I'm currently working on.
>
> However, it is self-contained, so it you attached the jar as well,
> users could try it if they wanted.
>
> Please remember to set the correct mime-type for any attachments.
>
> Also, it needs a license.
>
> S.
> On 10/06/06, Kumara Subramanya <ku...@gmail.com> wrote:
> > Hi,
> >
> > I wrote a plugin to enable sampling of mbean servers in jmeter. (refer
> > to bug 39642).
> > I added the source as an attachment to the issue in bugzilla.
> >
> >
> > Regards
> > Subrahmanya A
> > --
> > --
> > Subrahmanya Adiyanadka
> > --
> >
> > ---------------------------------------------------------------------
> > To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
> > For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
> >
> >
>
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
Re: The jmx sampler patch.
Posted by sebb <se...@gmail.com>.
Thanks, I saw that.
Unfortunately it's too late for the release I'm currently working on.
However, it is self-contained, so it you attached the jar as well,
users could try it if they wanted.
Please remember to set the correct mime-type for any attachments.
Also, it needs a license.
S.
On 10/06/06, Kumara Subramanya <ku...@gmail.com> wrote:
> Hi,
>
> I wrote a plugin to enable sampling of mbean servers in jmeter. (refer
> to bug 39642).
> I added the source as an attachment to the issue in bugzilla.
>
>
> Regards
> Subrahmanya A
> --
> --
> Subrahmanya Adiyanadka
> --
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
> For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org
>
>
---------------------------------------------------------------------
To unsubscribe, e-mail: jmeter-dev-unsubscribe@jakarta.apache.org
For additional commands, e-mail: jmeter-dev-help@jakarta.apache.org