Skip to content

feat(sync-v2): add needed storage indexes#226

Merged
jansegre merged 1 commit intodevfrom
feat/sync-v2-indexes
Jun 21, 2021
Merged

feat(sync-v2): add needed storage indexes#226
jansegre merged 1 commit intodevfrom
feat/sync-v2-indexes

Conversation

@jansegre
Copy link
Member

No description provided.

@jansegre jansegre self-assigned this Jun 11, 2021
@codecov-commenter
Copy link

Codecov Report

Merging #226 (c82b863) into dev (72b2473) will decrease coverage by 0.96%.
The diff coverage is 64.42%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev     #226      +/-   ##
==========================================
- Coverage   83.29%   82.33%   -0.97%     
==========================================
  Files         147      148       +1     
  Lines       14245    14608     +363     
  Branches     1996     2081      +85     
==========================================
+ Hits        11866    12028     +162     
- Misses       1952     2131     +179     
- Partials      427      449      +22     
Impacted Files Coverage Δ
hathor/p2p/manager.py 69.78% <ø> (ø)
hathor/manager.py 67.94% <49.33%> (-10.77%) ⬇️
hathor/p2p/downloader.py 78.74% <50.00%> (ø)
hathor/transaction/storage/transaction_storage.py 86.28% <63.08%> (-10.48%) ⬇️
hathor/transaction/base_transaction.py 92.76% <78.57%> (-0.53%) ⬇️
hathor/transaction/storage/block_height_index.py 80.76% <80.76%> (ø)
hathor/consensus.py 93.82% <100.00%> (-0.14%) ⬇️
hathor/daa.py 95.45% <100.00%> (+0.12%) ⬆️
hathor/p2p/node_sync.py 82.95% <100.00%> (ø)
hathor/transaction/resources/tips_histogram.py 100.00% <100.00%> (ø)
... and 16 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 72b2473...c82b863. Read the comment docs.

@jansegre jansegre force-pushed the feat/sync-v2-indexes branch 2 times, most recently from 9f5a900 to c75d5d3 Compare June 17, 2021 19:19
@jansegre jansegre force-pushed the feat/sync-v2-indexes branch 2 times, most recently from 0cf712c to 4ff2401 Compare June 18, 2021 03:12
@jansegre jansegre force-pushed the feat/sync-v2-indexes branch 4 times, most recently from a9c2489 to 8203311 Compare June 18, 2021 05:38
@jansegre jansegre force-pushed the feat/sync-v2-indexes branch from 8203311 to 5373415 Compare June 18, 2021 06:22
@jansegre jansegre force-pushed the feat/sync-v2-indexes branch from 5b2af49 to 401eed8 Compare June 21, 2021 18:50
@jansegre jansegre force-pushed the feat/sync-v2-indexes branch from 401eed8 to b146d3b Compare June 21, 2021 18:56
@jansegre jansegre merged commit 1bb29a5 into dev Jun 21, 2021
@jansegre jansegre deleted the feat/sync-v2-indexes branch June 21, 2021 19:09
@jansegre jansegre mentioned this pull request Jun 23, 2021
msbrogli added a commit that referenced this pull request Jul 2, 2021
This fixes a regression caused by PR #226.

Co-authored-by: Jan Segre <jan@hathor.network>
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.

4 participants