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: 🎸 Add process to batch block migrations #158

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

prashantasdeveloper
Copy link
Contributor

Description

With mapping handlers, all the old blocks were tried to be processed to be mapped in newer events in a single transaction. This even though was possible was keeping the block processing thread to remain open and also used to timeout if number of processed blocks was very high (~ 6M which needed almost 3600 timeout that the subquery-node is currently defaulted to).

This adds a max permissible block limit for migrating events data into entities to prevent timeout of block processing service. So, it fetches a set of blocks against every new block and migrates them into apt data. This will continue until all the old blocks are processed.

Breaking Changes

JIRA Link

Checklist

  • Updated the Readme.md (if required) ?

This adds a max permissible block limit for migrating events data into
entities to prevent timeout of block processing service.
@sonarcloud
Copy link

sonarcloud bot commented Jul 31, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@VictorVicente VictorVicente merged commit bba2b06 into alpha Aug 2, 2023
4 checks passed
@VictorVicente VictorVicente deleted the feat/batching-block-migration branch August 2, 2023 12:37
@polymath-eric
Copy link
Contributor

🎉 This PR is included in version 9.7.0-alpha.8 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@polymath-eric
Copy link
Contributor

🎉 This PR is included in version 9.8.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants