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.

Sign up for GitHub

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

Jump to bottom

Don't bind Transport before setting up aggregation and subscription phase #51

Closed
artem-v opened this issue Oct 30, 2018 · 1 comment
Closed

Don't bind Transport before setting up aggregation and subscription phase #51

artem-v opened this issue Oct 30, 2018 · 1 comment
Assignees
@artem-v
Labels
bug Something isn't working

Comments

@artem-v
Copy link
Contributor

artem-v commented Oct 30, 2018 •
edited

At the moment the bootstrap flow is like this: bind transport start listening -> activate subscribers -> do initial sync . The problem is that transport can accept incoming msgs even though not all subscribers were being set yet.
@artem-v artem-v added the bug Something isn't working label Oct 30, 2018
@artem-v artem-v self-assigned this Oct 30, 2018
@artem-v
Copy link
Contributor Author

artem-v commented Oct 30, 2018

Closing. Not relevant.

@artem-v artem-v closed this as completed Oct 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@artem-v artem-v

Labels
bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@artem-v