wrapped_domain table empty

RicqCodes / ens_migration_subsquid

A task to migrate ens sub-graph to sub-squid sdk

MIT License

0 stars 0 forks source link

wrapped_domain table empty #4

Open abernatskiy opened 1 year ago

abernatskiy commented 1 year ago

In a fully synced squid it has no records. Please fix.

RicqCodes commented 1 year ago

Thank you for feedback, I already fixed this in my latest commit

RicqCodes commented 1 year ago

I meant i fixed it in my last commit which i made recently after you spotted the bug!

abernatskiy commented 1 year ago

Getting a

QueryFailedError: insert or update on table "wrapped_domain" violates foreign key constrain

error now around the height of 16931400. Likely you insert a WrappedDomain entity record before inserting the Account entity it refers to.

RicqCodes commented 1 year ago

Thanks for providing the height, working on fixing it right now

RicqCodes commented 1 year ago

fixed with the recent commit

RicqCodes commented 11 months ago

i'm sure i have it fixed now

abernatskiy commented 11 months ago

Last time I tried syncing it on 11/10 I got a bunch of errors that suggested inserting entities out of order. I saw you've made new commits since then. Gonna take a look at what it does now.

RicqCodes commented 11 months ago

Sorry for all the headaches.

abernatskiy commented 11 months ago

The squid syncs now, but the name field for all WrappedDomain entity records is null. I found a bunch more issues:

Only 7 NewTTLs records in the squid vs 10+ in the subgraph
Only 2 AbiChanged records in the squid vs 10+ in the subgraph
No InterfaceChanged records in the squid vs 10+ in the subgraph
No NameUnwrapped records in the squid vs 10+ in the subgraph
No FusesSet records in the squid vs 10+ in the subgraph
For NameWrapped records the name field is null everywhere

Please fix.

You find this tool to be useful. Take a look at the (somewhat lax) comparison between the subgraph API and the squid API (my deployment of your code as it was on 17.11.2023) by running it as

npx compare-squid-to-subgraph https://api.thegraph.com/subgraphs/name/ensdomains/ens https://squid.subsquid.io/ricqcodes-ens-abernatskiy-test/v/v1/graphql -r 10 --temporal-ignore-ids --non-temporal-lower-case-ids

in the tool repo.

RicqCodes commented 11 months ago

Thank you very much, will fix this asap

RicqCodes commented 11 months ago

I'm sorry for silence, i have been trying to sync. I have a terrible internet over here so it's taking long time. It's been 4 days now

RicqCodes commented 11 months ago

Fixed bug with recent commit. All previous noted bugs has being fixed

abernatskiy commented 11 months ago

Thanks, testing! The squid takes a while to sync even on our infrastructure. I appreciate your effort.

RicqCodes commented 11 months ago

I totally understand i apologize for all the stress, syncing my buggy code while building an infrastructure. Thank you for always replying!

abernatskiy commented 11 months ago

Hello again! Your squid's API has passed the test. Congrats!

RicqCodes commented 11 months ago

Finally, thank you so much!