Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Improves JMS component robustness #240

Closed
wants to merge 3 commits into from
Closed

Conversation

max3163
Copy link
Contributor

@max3163 max3163 commented Dec 14, 2016

For both publisher and subcriber : improve error handling and allow to force reconnection against some error codes (using a new UI field and regular expression)

Only for subscriber, adds a wait between errors to avoid UI freeze due to infinite loops in case of repetitive errors.

PS : Some code relies on Java 8 (Predicate & lambda)

@loganmzz
Copy link

loganmzz commented Dec 19, 2016

Here is a sample script to check robustness against an ActiveMQ instance.

JMeter-ActiveMQ.zip

Try to launch script with a started/stopped server, and stop/restart server during test to check sample results and JMeter behaviour.

@pmouawad
Copy link
Contributor

Thanks for contribution.
Few notes:

  • docs (component-reference.xml) should be updated
  • In handleError, I think you should surround close line 232 with try/catch

@loganmzz
Copy link

Close publisher isn't supposed to throw exception. A quick view at the code, confirms only system (in VM sense) error can be raised. What is supposed to be done in this case ?

@asfgit asfgit closed this in 188d412 Jan 14, 2017
@pmouawad
Copy link
Contributor

Hello,
Thanks for PR.
I have commited it but with some slight changes.
Could you test nightly build and give us feedback ?
Also could you attach the JMX plan that you attached to this PR as a zip (which I am not able to read) to the bugzilla 60585

Thank you

@max3163 max3163 deleted the POW-314 branch April 27, 2017 17:33
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.

4 participants