Skip to content

Feat/dtynn/support more spec policy in assign unpacked deals#132

Merged
zl03jsj merged 7 commits intomainfrom
feat/dtynn/support_more_spec_policy_in_assign_unpacked_deals
May 14, 2022
Merged

Feat/dtynn/support more spec policy in assign unpacked deals#132
zl03jsj merged 7 commits intomainfrom
feat/dtynn/support_more_spec_policy_in_assign_unpacked_deals

Conversation

@dtynn
Copy link
Contributor

@dtynn dtynn commented May 13, 2022

  • 移除 AssignUnPackedDeals 接口中不必要的前置过滤逻辑
  • 处理 MinPiece, MinPieceSize, MinUsedSpace 三种新增策略,并搭配相应的测试用例
  • 为升级到 venus@master 增补 mock method

本 pr 应该被 Venus pr #4885 所阻塞,并应当在该pr合并后更新模块依赖

@zl03jsj
Copy link
Contributor

zl03jsj commented May 13, 2022

这里应该是要把把对Venus的引用升级到:
19620ae8f08164da72fb70885fcd1c01ef71f60d

@zl03jsj
Copy link
Contributor

zl03jsj commented May 14, 2022

我认为没有太大的问题, 提了一些小的修改意见, @dtynn

Copy link
Contributor

@zl03jsj zl03jsj left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

itgm

@zl03jsj zl03jsj merged commit 2c12f93 into main May 14, 2022
@zl03jsj zl03jsj deleted the feat/dtynn/support_more_spec_policy_in_assign_unpacked_deals branch May 14, 2022 04:41
litai686 pushed a commit to litai686/venus-market that referenced this pull request Jul 30, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants