Skip to content

fix: remove duplicate block store in engine_newPayloadV3 + update earliest and latest block number when storing genesis#245

Merged
fmoletta merged 1 commit into
mainfrom
minor-fixes
Aug 13, 2024
Merged

fix: remove duplicate block store in engine_newPayloadV3 + update earliest and latest block number when storing genesis#245
fmoletta merged 1 commit into
mainfrom
minor-fixes

Conversation

@fmoletta

@fmoletta fmoletta commented Aug 12, 2024

Copy link
Copy Markdown
Contributor

Motivation

Fixes two small issues:

  1. As the execute_block function now also stores the block in the db the call to add_block in engine_newPaylaodV3 is redundant
  2. Inability to fetch lates block when no block other than the genesis block has been executed

Description

  • Remove call to add_block in engine_newPaylaodV3
  • Store the genesis block number as earliest and latest block number in the db in Store::add_initial_state

@fmoletta fmoletta changed the title fix: remove duplicate block store in engine_newPayloadV3 + update earliest and latest block number when storing genes fix: remove duplicate block store in engine_newPayloadV3 + update earliest and latest block number when storing genesis Aug 12, 2024
@fmoletta fmoletta marked this pull request as ready for review August 12, 2024 19:47
@fmoletta fmoletta requested a review from a team as a code owner August 12, 2024 19:47
@fmoletta fmoletta added this pull request to the merge queue Aug 13, 2024
Merged via the queue into main with commit 976cad4 Aug 13, 2024
@fmoletta fmoletta deleted the minor-fixes branch August 13, 2024 13:41
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.

2 participants