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

fix problems with Java Node Coordination example #537

Merged
merged 1 commit into from
Mar 26, 2015

Conversation

calid
Copy link
Contributor

@calid calid commented Mar 26, 2015

There were two problems where the java subscribers were receiving a low
message count and also not always exiting (implying they weren't
receiving the END message).

Setting a high receive hwm (unlimited for example simplicity) seems to
have resolved both issues, and the subscribers now both receive the full
complement of messages as well as exit correctly.

For reference: https://lists.zeromq.org/pipermail/zeromq-dev/2015-March/028585.html
My guess is all the bindings showing problematic behavior can probably be fixed by tweaking receive hwm.

Thanks!

There were two problems where the java subscribers were receiving a low
message count and also not always exiting (implying they weren't
receiving the END message).

Setting a high receive hwm (unlimited for example simplicity) seems to
have resolved both issues, and the subscribers now both receive the full
complement of messages as well as exit correctly.
hintjens added a commit that referenced this pull request Mar 26, 2015
fix problems with Java Node Coordination example
@hintjens hintjens merged commit 9b0112c into booksbyus:master Mar 26, 2015
@calid calid deleted the fix-java-sync-example branch March 27, 2015 09:14
calid added a commit to calid/zguide that referenced this pull request Mar 28, 2015
Missing messages and/or subscribers not exiting.

setting high publish sndhwm (unlimited for example simplicity) was
sufficient to fix the issues in all these languages

ref: booksbyus#537
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.

2 participants