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

Feature/network refactor #1804

Merged
merged 7 commits into from
Dec 15, 2020
Merged

Feature/network refactor #1804

merged 7 commits into from
Dec 15, 2020

Conversation

jolestar
Copy link
Member

@jolestar jolestar commented Dec 11, 2020 •
edited
Loading

  1. 简化 Network 的消息处理机制，通过 ActorService 托管 NetworkWorker future.
  2. Peers 中不在包含当前节点自己。
  3. 在网络层过滤重复的消息。
  4. 增加更多测试。

@jolestar jolestar force-pushed the feature/network_refactor branch from 4cf7574 to 59e4ef4 Compare December 11, 2020 15:26
@jolestar jolestar marked this pull request as draft December 11, 2020 15:32
@jolestar jolestar force-pushed the feature/network_refactor branch from d0d405d to 802c486 Compare December 13, 2020 02:40
@jolestar jolestar marked this pull request as ready for review December 13, 2020 02:47
@jolestar jolestar force-pushed the feature/network_refactor branch from 8d48350 to ac5a993 Compare December 13, 2020 11:02
@jolestar jolestar merged commit 77a17fa into master Dec 15, 2020
@jolestar jolestar deleted the feature/network_refactor branch December 15, 2020 06:34
@jolestar jolestar mentioned this pull request Jan 5, 2021
Closed
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sanlee42 sanlee42 sanlee42 approved these changes

@tiangong3624749 tiangong3624749 Awaiting requested review from tiangong3624749

@fanngyuan fanngyuan Awaiting requested review from fanngyuan

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jolestar @sanlee42