No records in dex_aggregator.trades about spellbook

[jeff-dude]

thanks for raising.

looks like we never cleaned up full path for odos, as we had it excluded previously due to incorrect logic and prod failures. see the final dex_aggregator spell here:
https://github.com/duneanalytics/spellbook/blob/main/models/dex/dex_aggregator_trades.sql#L18-L21
and how only optimism is included:
https://github.com/duneanalytics/spellbook/blob/main/models/dex/dex_aggregator_trades.sql#L37

feel free to open PR to clean that up 👍
you can also add base to odos_trades.sql?

[amalashkevich]

@jeff-dude thanks for the info.
I think I did it here #5718
But the test fails, not sure if this is my change, could you take a look?
https://github.com/duneanalytics/spellbook/actions/runs/8526776877/job/23356758978?pr=5718

[amalashkevich]

Hi @jeff-dude
How can I see how the line 420 looks like? It says "Column 'block_month' cannot be resolved", but according to the schema, the column name 'block_month' is there.

13:47:25 Database Error in model dex_aggregator_trades (models/dex/dex_aggregator_trades.sql)
TrinoUserError(type=USER_ERROR, name=COLUMN_NOT_FOUND, message="line 420:7: Column 'block_month' cannot be resolved", query_id=20240404_134621_00008_qupjv)
compiled Code at target/run/spellbook/models/dex/dex_aggregator_trades.sql

[jeff-dude]

let's complete this in your PR