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

Streaming - Experimental Support #90

Closed
wants to merge 9 commits into from
Closed

Streaming - Experimental Support #90

wants to merge 9 commits into from

Conversation

mmmries
Copy link
Collaborator

@mmmries mmmries commented May 10, 2019

This is a first second third pass at streaming support. It follows a pattern of using get_statem process that finds a gnat connection, tries to register itself and then monitors the connection so that it can reset the state machine if the connection dies.

For example, here is a state diagram for a subscription

Screen Shot 2019-05-10 at 8 22 29 AM

In future PRs I'm imagining a higher-level supervisor that will automatically setup the connection, streaming client and subscribers by adding a single supervisor to a user's app.

@mmmries
Copy link
Collaborator Author

mmmries commented Jun 1, 2019

I was feeling stuck with how to release this additional support when I am still very un-sure of whether this is the right API I was users to use, so I've decided to close this PR and start a separate gnat_streaming package that will handle all things streaming. That way I can release that as a 0.0.1 version and make it very clear that it is a alpha/beta version of the code

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.

1 participant