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

Long timeout in receiving can prevent the background handling from other message streams #41

Open
jussimalinen opened this issue Dec 1, 2015 · 2 comments
Labels
Milestone

Comments

@jussimalinen
Copy link
Contributor

Receiving with timeout goes to a synchronized block. If the timeout is longer than the poll interval for background handlers, this can cause the other message streams to not get the lock during this time and they can not handle their background messages.

The receiving of protocol headers in MessageStream should be done so that other message streams either can operate at the same time, or can get the lock periodically.

vivektatipamula pushed a commit to vivektatipamula/Rammbock that referenced this issue Dec 4, 2015
@jussimalinen jussimalinen added this to the 0.5.0 milestone Dec 4, 2015
vivektatipamula pushed a commit to vivektatipamula/Rammbock that referenced this issue Dec 9, 2015
…tance tests accordingly. Added new atest to test new code. MarketSquare#41
@niravsh
Copy link

niravsh commented Dec 22, 2015

Is there a work around for this bug ? I believe I am facing this issue of tests stuck for long time.

vivektatipamula added a commit to vivektatipamula/Rammbock that referenced this issue Jan 15, 2016
@vivektatipamula
Copy link

@niravsh , this fix should help you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants