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

feat:support get deals by datacid from database but not only rely on topindex #160

Merged
merged 1 commit into from
Jul 5, 2022

Conversation

hunjixin
Copy link
Contributor

@hunjixin hunjixin commented Jul 5, 2022

@codecov-commenter
Copy link

Codecov Report

Merging #160 (e636216) into main (587bd0a) will increase coverage by 0.27%.
The diff coverage is 32.89%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #160      +/-   ##
==========================================
+ Coverage   18.81%   19.08%   +0.27%     
==========================================
  Files         170      171       +1     
  Lines       17804    17954     +150     
==========================================
+ Hits         3349     3426      +77     
- Misses      14023    14087      +64     
- Partials      432      441       +9     
Flag Coverage Δ
unittests 19.08% <32.89%> (+0.27%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
models/mysql/storage_deal.go 0.00% <0.00%> (ø)
models/repo/repo.go 100.00% <ø> (ø)
dagstore/mock_dagstore_wrapper.go 21.17% <21.17%> (ø)
models/badger/storage_deal.go 36.62% <65.38%> (+9.89%) ⬆️
retrievalprovider/piecestore.go 73.17% <83.33%> (+73.17%) ⬆️
paychmgr/simple.go 79.36% <0.00%> (-1.14%) ⬇️
blockstore/badger/blockstore.go 56.36% <0.00%> (+0.45%) ⬆️

@diwufeiwen diwufeiwen merged commit ad6cbb4 into main Jul 5, 2022
@diwufeiwen diwufeiwen deleted the feat/support_retrieval_by_data_cid branch July 5, 2022 05:28
litai686 pushed a commit to litai686/venus-market that referenced this pull request Jul 30, 2022
…/feat/dtynn/detection_of_sector_state_missing

Feat/dtynn/detection of sector state missing
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.

[venus-market] 检索内置支持通过data root进行检索
3 participants