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

Simple latched tcp topics #3

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

peci1
Copy link
Contributor

@peci1 peci1 commented Oct 29, 2016

The simpler version. Still don't know which one to choose... This one needs a bit more documentation.

@xqms
Copy link
Member

xqms commented Oct 30, 2016

I think we still have to solve the situation that the receiver starts after the sender, but there are no new messages. In this situation, connect() is not called, and latched messages are not sent.

I would propose starting a "connection" timer, that tries to connect (if there is no connection).
To reliably detect connection loss (e.g. the receiver was restarted), we should also probably set the SO_KEEPALIVE socket option.

This whole machinery should only be active if we actually have latched topics.

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