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

BlocksTransactionsMismatch CPU fix #2109

Merged
merged 3 commits into from
Jun 10, 2019

Conversation

pasqu4le
Copy link
Contributor

@pasqu4le pasqu4le commented Jun 6, 2019

Motivation

Due to heavy CPU utilization this PR changes the update queries to avoid the problem.

Changelog

Enhancements

The changes make it so to use 2 "big" updates instead of Multi transactions in BlocksTransactionsMismatch temporary fetcher

Checklist for your PR

  • I added an entry to CHANGELOG.md with this PR
  • If I added new functionality, I added tests covering it.
  • If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
  • I checked whether I should update the docs and did so if necessary

@pasqu4le pasqu4le requested a review from vbaranov June 6, 2019 13:27
@pasqu4le pasqu4le self-assigned this Jun 6, 2019
@coveralls
Copy link

coveralls commented Jun 6, 2019

@pasqu4le pasqu4le added waiting on review Reviewer is analyzing the issue. ready for review This PR is ready for reviews. and removed in progress waiting on review Reviewer is analyzing the issue. labels Jun 7, 2019
@vbaranov vbaranov merged commit d68ad0d into master Jun 10, 2019
@vbaranov vbaranov deleted the blocks_transactions_mismatch_cpu_fix branch June 10, 2019 12:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

6 participants