graph-node icon indicating copy to clipboard operation
graph-node copied to clipboard

Wrong event order on same block from Aave clone V2 on Godwoken chain

Open Roger-RumbleFish opened this issue 3 years ago • 1 comments

Here is my subgraph repo -> https://github.com/Roger-RumbleFish/Subgraph-test

the issue is between createOrLoadReserve and CollateralConfigurationChangedHandler

they should be called in order

  1. createOrLoadReserve (all 6 tokens)
  2. CollateralConfigurationChangedHandler (all 6 tokens)

Unfortunately some of the events are called not in correct order. Because my events are depend on each other i have null values for wBTC reserve

image

the problem is between blocks 72543 -> 72544 when i deployed the contracts

When i make getLogs call i get them in correct order

image (9)

if you need the rpc for godwoken mainnet here it is -> https://v1.mainnet.godwoken.io/rpc

to run it

  1. yarn
  2. docker-compose up
  3. yarn rebuild

Roger-RumbleFish avatar Aug 26 '22 10:08 Roger-RumbleFish

Hi - it is not clear to me that there is an ordering issue here, indeed I think there shouldn't be. Can you provide Graph Node logs to clarify the exact order of execution? You can add logging to help with debugging

azf20 avatar Sep 01 '22 11:09 azf20

Hi, @Roger-RumbleFish Thanks for the bug report. It was a bug in Godwoken web3-indexer which has been fixed.

PR

[Log's index should index in a block, not transaction]

  • https://github.com/godwokenrises/godwoken-web3/pull/553)

Releases

  • https://github.com/godwokenrises/godwoken-info/pull/69
  • https://github.com/godwokenrises/godwoken-web3/releases/tag/v1.8.5 (Godwoken testnet_v1)
  • https://github.com/godwokenrises/godwoken-web3/releases/tag/v1.7.4 (Godwoken mainnet_v1)

Flouse avatar Oct 20 '22 05:10 Flouse

Thanks!

azf20 avatar Oct 24 '22 11:10 azf20