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

[venus-messager] 消息增加uuid? #5203

Closed
1 of 12 tasks
Fatman13 opened this issue Aug 19, 2022 · 3 comments
Closed
1 of 12 tasks

[venus-messager] 消息增加uuid? #5203

Fatman13 opened this issue Aug 19, 2022 · 3 comments
Labels
C-enhancement Category: feature enhancement design Venus feature/system design issues

Comments

@Fatman13
Copy link
Contributor

链服务模块 / Chain Service Components

  • venus
  • venus-auth
  • venus-gateway
  • venus-messager
  • venus-miner
  • 文档 / docs

订单服务模块 / Deal Service Components

  • venus-market
  • 文档 / docs

算力服务模块 / Storage Power Service Components

  • venus-sector-manager
  • venus-worker
  • 文档 / docs

描述 / Description

待评估,venus可能不适用;Lotus给miner发送的消息带上uuid,不知道是否需要借鉴,还是完全不适用。

关联:

@Fatman13 Fatman13 added the C-enhancement Category: feature enhancement label Aug 19, 2022
@Fatman13 Fatman13 added the design Venus feature/system design issues label Aug 19, 2022
@hunjixin
Copy link
Contributor

神奇这个东西。 我觉得有这个还不如让他们考虑往我们messager里面推送。

@Fatman13
Copy link
Contributor Author

Fatman13 commented Aug 19, 2022

哈哈哈。Lotus变得越来越Venus化?

@Fatman13 Fatman13 added the C-triage Category: Need triage label Aug 22, 2022
@diwufeiwen
Copy link
Contributor

diwufeiwen commented Aug 25, 2022

这个与venus-messager没有关系,在lotus中对于已签名消息有一个k-v数据库存取,这个uuid就是生成Key的,规则: "MsgUuidSet-uuid.String()"。这个在我们的体系中没必要引入。我们venus-messager有自己的机制排查重复的消息

@diwufeiwen diwufeiwen mentioned this issue Aug 25, 2022
33 tasks
@diwufeiwen diwufeiwen moved this to Done in Venus Project Aug 25, 2022
@diwufeiwen diwufeiwen removed the C-triage Category: Need triage label Aug 26, 2022
@Fatman13 Fatman13 closed this as not planned Won't fix, can't repro, duplicate, stale Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category: feature enhancement design Venus feature/system design issues
Projects
Archived in project
Development

No branches or pull requests

3 participants