-
Notifications
You must be signed in to change notification settings - Fork 109
feat: implement indexer processors v2 #1264
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
I've made an attempt to switch out the old indexer processors for v2; it compiles but when I run it with
I checked and it doesn't seem like the database is already running... so I'm not sure why there would be duplicate key values. Tip I think it may be because each processor is trying to run a migration... so maybe what I need to do is run a migration once and then run the processors. |
Now that I seem to have perhaps resolved the Stack backtrace with
|
I think it's because we don't execute the consensus, our Tx misses some data, for example, the WriteSet. Can you look at the code where there is None to see which data is missing? |
@musitdev the first one is at
and there's also
Yes I'll try it that way with the devnet DB. |
Ok, the movement node doesn't set the table_item, that's why I've deactivated them on the indexer v1. For the migration, you can not do that; you need to use an Aptos node that will set the table_item. |
@musitdev so I'm starting the Aptos node with But when I run ![]() Am I supposed to copy and paste those Cc @areshand @0xmovses as well if you can provide some guidance on this... I'm trying to do what @musitdev mentions: "Start an Aptos node, then load a movement db with some tx then you can test the indexer v2." |
there has to be a db folder under .maptos. For example, the path has to be .maptos/db/ledger_db and .maptos/db/state_merkle_db. The db_dir you used for the aptos node_config is ..../.maptos (note: don't include in the "db" in path) |
Closing this as I've been able to run the V2 processors against |
Summary
Changelog
Testing
Outstanding issues
diesel
dependency version mismatch error preventing compilation:The indexer did not work on
main
so this branch is cut from thefeature/da-sequencer-node
branch, where the indexer does work.Depends on https://github.com/movementlabsxyz/aptos-core/pull/182/commits and chore: downgrade diesel version to 2.2.0 aptos-core#182 for
diesel
version 2.2.0.