Skip to content

Conversation

@mkalinin
Copy link
Contributor

Refines the Message ordering section of the Engine API.

The previous version of the spec had the following overlooks:

  • CL could flip two forkchoiceUpdated messages before the IDs were assigned which could result in inconsistency of the head of the chain between the layers
  • The restrictions on EL side were too strong. EL couldn't process two different messages in parallel which e.g. locks the client until executePayload processing is finished

Proposed changes:

  • CL: add the requirement to respect the order of occurrence of fork choice updated event
  • EL: restrict enforcement of processing order to forkchoiceUpdated calls only

Copy link
Member

@lightclient lightclient left a comment

Choose a reason for hiding this comment

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

1 nit, but generally LGTM.

@ethereum ethereum deleted a comment from kenyahewitt Nov 29, 2021
Co-authored-by: Danny Ryan <dannyjryan@gmail.com>
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.

3 participants