Skip to content

Conversation

@larskuhtz
Copy link
Contributor

@larskuhtz larskuhtz commented Oct 13, 2025

Ea has a race condition in the Pact transaction parser. It is not completely clear to me what triggers it.One possibility would be the generation of headers that contain information for more than one version.

Making Ea process chainweb versions sequentially probably fixes it.

@larskuhtz larskuhtz force-pushed the evm/lars/fix-race-in-ea branch from dc49fb1 to 1d9401b Compare October 13, 2025 04:40
@larskuhtz larskuhtz marked this pull request as ready for review October 13, 2025 05:17
@edmundnoble
Copy link
Contributor

We used to have more payloads being generated in ea, and this saved us a lot of time. If the parser throws an error can you post it?

Copy link
Contributor

@edmundnoble edmundnoble left a comment

Choose a reason for hiding this comment

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

I'm not convinced that this is a good idea. I don't see any possibility for races here. If anything, I'd like ea to have more concurrency, not less.

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