Closed m-Peter closed 1 month ago
The recent update in the Engine
struct's processEvents
method focuses on correcting the source of the EVM block height. By switching from events.CadenceHeight()
to events.Block().Height
, the change ensures that the reported height accurately reflects the EVM block height rather than the Cadence block height. This aligns the emitted metrics with the actual data from the EVM, resolving previous discrepancies.
Files | Change Summary |
---|---|
services/ingestion/engine.go | Updated processEvents method to use events.Block().Height for EVM height. |
Objective | Addressed | Explanation |
---|---|---|
EVM blockheight metric being emitted is Cadence block height (455) | β |
π° In the meadow, where data hops,
The EVM's height no longer flops.
With a switch so deft,
Cadence left bereft,
Now accurate heights, sweet as carrots, pop! πΏπ₯
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Closes: https://github.com/onflow/flow-evm-gateway/issues/455
Description
For contributor use:
master
branchFiles changed
in the Github PR explorerSummary by CodeRabbit