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.

Sign up for GitHub

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

Jump to bottom

[database] Update BadgerDB Performance Settings #267

Merged
merged 1 commit into from
Dec 5, 2020
Merged

[database] Update BadgerDB Performance Settings #267

merged 1 commit into from
Dec 5, 2020

Conversation

patrick-ogrady
Copy link
Contributor

@patrick-ogrady patrick-ogrady commented Dec 5, 2020 •
edited
Loading

To support blockchains with large blocks, we need to increase the table size configuration in storage/database.

Side Effects

This ONLY affects rosetta-cli when disable_memory_limit is set to true!

@patrick-ogrady patrick-ogrady changed the title [database] Update Performance Settings [database] Update BadgerDB Performance Settings Dec 5, 2020
@patrick-ogrady patrick-ogrady merged commit 0f2c511 into master Dec 5, 2020
@patrick-ogrady patrick-ogrady deleted the patrick/modify-performance-options branch December 5, 2020 00:41
@coveralls
Copy link

Pull Request Test Coverage Report for Build 13188

  • 0 of 2 (0.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.008%) to 78.372%
Changes Missing Coverage Covered Lines Changed/Added Lines %
storage/database/badger_database.go 0 2 0.0%
Totals Coverage Status
Change from base Build 13182: -0.008%
Covered Lines: 7932
Relevant Lines: 10121
💛 - Coveralls

@sidhujag sidhujag mentioned this pull request Dec 5, 2020
Closed
@patrick-ogrady patrick-ogrady mentioned this pull request Dec 7, 2020
Merged
2 tasks
@sidhujag sidhujag mentioned this pull request Dec 7, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@yfl92 yfl92 yfl92 approved these changes

Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@patrick-ogrady @coveralls @yfl92