Summary: | Adds static and dynamic destinations to JMS Publisher | ||
---|---|---|---|
Product: | JMeter - Now in Github | Reporter: | Jason Butler <jbutler> |
Component: | Main | Assignee: | JMeter issues mailing list <issues> |
Status: | RESOLVED FIXED | ||
Severity: | enhancement | ||
Priority: | P2 | ||
Version: | unspecified | ||
Target Milestone: | --- | ||
Hardware: | All | ||
OS: | Linux | ||
Attachments: |
patch file
Screenshot New screenshot |
Thanks for your patch. I works too on a JMS patch for dynamic/static publisher and subscriber. Your approach is different and seems better vs my (futur) patch. I will try to test and verify your patch the next week. Milamber Looks OK to me as well. One minor tweak to the patch - when getting/setting the property USE_STATIC_DEST, the default needs to be "true". Otherwise existing plans will be treated as dynamic, and saved "static" plans will have an unnecessary property. Note to self: need to update component_reference.xml as well. Forgot to add - no need to resubmit the patch - we can easily add the extra "default" parameter to the get/set Property method calls. OK for comments, sebb. Currently I works on this patch. I will change some GUI (came from my patch) for put the option in bottom (see screenshot) (comments on labels/type are welcome) Created attachment 26302 [details]
Screenshot
Sorry, but I don't like the position of the radio buttons so far from the Destination field. The dest field does not need to be the full width of the screen, so the selection field should fit on the same line. As to the actual text - if it is on the same line, it could be simplified somewhat; perhaps to: Destination | | Setup: o At startup o Each sample Created attachment 26306 [details]
New screenshot
Thanks for your comments.
In this screnshot, I have doing your suggestion
Thanks! Looks good. Done. I have tried to preserve API on Publisher.java URL: http://svn.apache.org/viewvc?rev=1036177&view=rev Log: Bug 50268 - Adds static and dynamic destinations to JMS Publisher Modified: jakarta/jmeter/trunk/src/core/org/apache/jmeter/resources/messages.properties jakarta/jmeter/trunk/src/core/org/apache/jmeter/resources/messages_fr.properties jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/client/Publisher.java jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/control/gui/JMSPublisherGui.java jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/BaseJMSSampler.java jakarta/jmeter/trunk/src/protocol/jms/org/apache/jmeter/protocol/jms/sampler/PublisherSampler.java jakarta/jmeter/trunk/xdocs/changes.xml jakarta/jmeter/trunk/xdocs/images/screenshots/jmspublisher.png jakarta/jmeter/trunk/xdocs/usermanual/component_reference.xml This issue has been migrated to GitHub: https://github.com/apache/jmeter/issues/2426 |
Created attachment 26295 [details] patch file This patch provides the ability for JMSProducer to change destinations while the test runs so that it works with variables and foreach controllers. The option to use the current behavior is still preserved. Adds a check box for static destinations to the JMSProducer sampler gui. If "Use static destination" is not checked then the destination the producer sends to will update each time the thread group loops. This way the producer can send to a different destination each time. If "Use static destination" is checked then the destination will be fixed the first time the thread group loops which is the current behavior.