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

[venus] 收到区块慢相对于louts普遍慢一秒左右 #5008

Closed
Tracked by #5257
simlecode opened this issue Jun 30, 2022 · 2 comments
Closed
Tracked by #5257

[venus] 收到区块慢相对于louts普遍慢一秒左右 #5008

simlecode opened this issue Jun 30, 2022 · 2 comments
Assignees
@simlecode
Labels
C-Blocked Category: temporarily stuck without a good solution P2 Medium - we should get to this soon V-venus venus component

Comments

@simlecode
Copy link
Collaborator

simlecode commented Jun 30, 2022 •
edited
Loading

经排查，venus 在 pubsub 层收到区块就比 lotus 慢，可以排除 pubsub 层以上代码的影响，后续可以对考虑连接的 peer 和 libp2p 进行研究

代码层面对比：

  1. 都有 ConnectionManager，参数也相差不大
  2. lotus 有 peer score管理，venus 加上也并没有明显改善收到区块慢的情况
@Fatman13 Fatman13 added the C-need-testing Category: Venus testing related label Jul 1, 2022
@hunjixin hunjixin changed the title [venus] 收到区块慢 [venus] 收到区块慢相对于louts普遍慢一秒左右 Jul 1, 2022
@hunjixin hunjixin added P2 Medium - we should get to this soon V-venus venus component labels Jul 1, 2022
@simlecode simlecode moved this to In Progress in Venus Project Jul 4, 2022
@diwufeiwen diwufeiwen mentioned this issue Jul 25, 2022
Closed
36 tasks
@simlecode simlecode added the C-Blocked Category: temporarily stuck without a good solution label Jul 25, 2022
@hunjixin hunjixin mentioned this issue Aug 8, 2022
Closed
25 tasks
@diwufeiwen diwufeiwen removed the status in Venus Project Aug 19, 2022
@hunjixin hunjixin mentioned this issue Sep 5, 2022
Closed
31 tasks
@zl03jsj zl03jsj self-assigned this Sep 5, 2022
@zl03jsj zl03jsj moved this to In Progress in Venus Project Sep 6, 2022
@simlecode
Copy link
Collaborator Author

image
上图是用 chain-co 来连接venus和lotus，再通过比较head的变化来对比收到区块的时间，相同高度等head稳定所花时间相差不大

@zl03jsj zl03jsj removed their assignment Sep 15, 2022
Repository owner moved this from In Progress to Done in Venus Project Sep 16, 2022
@simlecode simlecode mentioned this issue Sep 16, 2022
Closed
@YiRuiLiher
Copy link
Collaborator

1

@Fatman13 Fatman13 removed the C-need-testing Category: Venus testing related label Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@simlecode simlecode

Labels
C-Blocked Category: temporarily stuck without a good solution P2 Medium - we should get to this soon V-venus venus component
Projects
Venus Project
Archived in project
Milestone
No milestone
Development

No branches or pull requests
5 participants
@Fatman13 @zl03jsj @hunjixin @YiRuiLiher @simlecode