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

JMSBroadcaster is misusing JMS session objects #161

Closed
mattnathan opened this issue Jan 23, 2012 · 3 comments
Closed

JMSBroadcaster is misusing JMS session objects #161

mattnathan opened this issue Jan 23, 2012 · 3 comments

Comments

@mattnathan
Copy link
Contributor

The JMS spec says that if you have a thread that is both a consumer and a producer of messages then you need a session for each task. In JMSBroadcaster a single session is being shared for both these tasks. When using WebSphere MQ this causes an exception when creating the TextMessage for publishing.

The simple fix is to create two sessions in the setUp method one for publishing and one for consuming.

@mattnathan
Copy link
Contributor Author

See here for an example fix for this issue: mattnathan@8fcb850

@jfarcand
Copy link
Member

The fix look good to me but I don't have the setup to test it. If that works for you, just do a pull request and I will integrate in the current 0.9-SNAPSHOT

@jfarcand
Copy link
Member

Fixed, thanks!

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

No branches or pull requests

2 participants