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(core-database): add block and transactions indexes migration #3892

Merged
merged 4 commits into from
Jul 15, 2020

Conversation

sebastijankuzner
Copy link
Contributor

@sebastijankuzner sebastijankuzner commented Jul 13, 2020

Summary

This PR solves: #3890

It adds block and transaction indexes.

Checklist

  • Ready to be merged

@ghost ghost added the Complexity: Medium label Jul 13, 2020
@sebastijankuzner sebastijankuzner changed the title Feat/core database/migrations feat(core-database): add block and transactions indexes migration Jul 13, 2020
@codecov
Copy link

codecov bot commented Jul 13, 2020

Codecov Report

Merging #3892 into develop will increase coverage by 3.86%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop    #3892      +/-   ##
===========================================
+ Coverage     9.87%   13.73%   +3.86%     
===========================================
  Files          618      620       +2     
  Lines        14433    14445      +12     
  Branches      1725     1726       +1     
===========================================
+ Hits          1425     1984     +559     
+ Misses       12902    12345     -557     
- Partials       106      116      +10     
Flag Coverage Δ
#functional 6.79% <100.00%> (?)
#integration 9.87% <0.00%> (-0.01%) ⬇️
Impacted Files Coverage Δ
...0000-add-type_group-index-to-transactions-table.ts 100.00% <100.00%> (ø)
...00317000000-add-blocks-and-transactions-indexes.ts 100.00% <100.00%> (ø)
packages/core-p2p/src/service-provider.ts 0.00% <0.00%> (ø)
packages/core-p2p/src/peer-communicator.ts 0.00% <0.00%> (ø)
packages/core-p2p/src/peer.ts 61.53% <0.00%> (+3.20%) ⬆️
...ges/core-test-framework/src/app/generators/core.ts 73.33% <0.00%> (+6.66%) ⬆️
packages/core-test-framework/src/utils/generic.ts 68.96% <0.00%> (+27.58%) ⬆️
...s/core-database/src/utils/snake-naming-strategy.ts 36.36% <0.00%> (+36.36%) ⬆️
... and 39 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 381e9c7...8be1263. Read the comment docs.

@air1one air1one merged commit b42bfb6 into develop Jul 15, 2020
@ghost ghost deleted the feat/core-database/migrations branch July 15, 2020 12:36
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.

None yet

2 participants