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] venus 和 venus-market 合并 blockstore 相关实现 #5138

Closed
2 of 11 tasks
simlecode opened this issue Jul 27, 2022 · 3 comments · Fixed by ipfs-force-community/droplet#204
Closed
2 of 11 tasks
Assignees
Labels
C-Blocked Category: temporarily stuck without a good solution C-enhancement Category: feature enhancement design Venus feature/system design issues Epic

Comments

@simlecode
Copy link
Collaborator

simlecode commented Jul 27, 2022

链服务模块 / 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 和 venus-market 都各自实现了一套 blockstore 相关的实现,可以考虑合并到一起

venus 代码在:https://github.com/filecoin-project/venus/tree/master/pkg/util/blockstoreutil
venus-market 代码在:https://github.com/filecoin-project/venus-market/tree/master/blockstore

@simlecode simlecode added the C-enhancement Category: feature enhancement label Jul 27, 2022
@Fatman13 Fatman13 added Epic design Venus feature/system design issues labels Jul 29, 2022
@hunjixin
Copy link
Contributor

@simlecode 已解决?

@simlecode
Copy link
Collaborator Author

待做

@hunjixin
Copy link
Contributor

hunjixin commented Nov 2, 2022

@LinZexiao 可以合并了?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Blocked Category: temporarily stuck without a good solution C-enhancement Category: feature enhancement design Venus feature/system design issues Epic
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants