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

Network thread refactor #807

Closed
neuroscr opened this issue Sep 4, 2019 · 4 comments
Closed

Network thread refactor #807

neuroscr opened this issue Sep 4, 2019 · 4 comments
Assignees
@majestrate @michael-loki
Labels
enhancement New feature or request
Projects
Enhancements
Milestone
0.6

Comments

@neuroscr
Copy link
Contributor

neuroscr commented Sep 4, 2019

figure out how to have a network thread to coordinate with the rest of the system in a performant way, to enable Tom's prioritization work.
@neuroscr neuroscr added the enhancement New feature or request label Sep 4, 2019
@neuroscr neuroscr added this to the 0.6 milestone Sep 4, 2019
@neuroscr neuroscr added this to To do in Enhancements via automation Sep 4, 2019
@neuroscr neuroscr mentioned this issue Sep 5, 2019
Closed
@neuroscr
Copy link
Contributor Author

main thread is the network thread. Everything is moved out (crypto, logic)

@neuroscr
Copy link
Contributor Author

queueing to the network is done in the logic thread. network rn just reads/writes to sockets.

@neuroscr
Copy link
Contributor Author

done in jeffs branch, jeff will PR soon

@neuroscr
Copy link
Contributor Author

neuroscr commented Oct 9, 2019

PR is merged

@neuroscr neuroscr closed this as completed Oct 9, 2019
Enhancements automation moved this from To do to Done Oct 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@majestrate majestrate

@michael-loki michael-loki

Labels
enhancement New feature or request
Projects
No open projects
Enhancements
  
Done
Milestone
0.6
Development

No branches or pull requests
3 participants
@majestrate @neuroscr @michael-loki