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

Freezer fix #2213

Merged
merged 1 commit into from
Feb 20, 2024
Merged

Freezer fix #2213

merged 1 commit into from
Feb 20, 2024

Conversation

MatusKysel
Copy link
Contributor

@MatusKysel MatusKysel commented Feb 14, 2024

Description

This is cherry-picked commit from geth, that fsync the index and data file after each freezer write and should resolve #2209

Rationale

tell us why we need these changes...

Example

add an example CLI or API response...

Changes

Notable changes:

  • add each change in a bullet point here
  • ...

* core/rawdb: fsync the index and data file after each freezer write

* core/rawdb: fsync the data file in freezer after write
@MatusKysel MatusKysel changed the base branch from master to develop February 14, 2024 07:23
@MatusKysel MatusKysel requested a review from zzzckck February 19, 2024 07:28
@MatusKysel MatusKysel marked this pull request as ready for review February 19, 2024 07:28
Copy link
Collaborator

@zzzckck zzzckck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@MatusKysel MatusKysel merged commit 40cae45 into develop Feb 20, 2024
6 checks passed
@MatusKysel MatusKysel deleted the freezer-fix branch February 20, 2024 08:30
@zzzckck zzzckck mentioned this pull request Feb 20, 2024
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