Add poly, arb, and bnb xchange dex trades models and include xchange trades in dex.trades

[mike-x7f]

Brief comments on the purpose of your changes:

• Create new source, schema, model for xchange trades on arbitrum, polygon, and bnb chains
• Create main xchange_trades model which unions xchange trades from all chains
• Add xchange_trades to dex_trades model

For Dune Engine V2

I've checked that:

General checks:

• I tested the query on dune.com after compiling the model with dbt compile (compiled queries are written to the target directory)
• I used "refs" to reference other models in this repo and "sources" to reference raw or decoded tables
• if adding a new model, I added a test
• the filename is unique and ends with .sql
• each sql file is a select statement and has only one view, table or function defined
• column names are lowercase_snake_cased
• if adding a new model, I edited the dbt project YAML file with new directory path for both models and seeds (if applicable)
• if wanting to expose a model in the UI (Dune data explorer), I added a post-hook in the JINJA config to add metadata (blockchains, sector/project, name and contributor Dune usernames)

Pricing checks:

• coin_id represents the ID of the coin on coinpaprika.com
• all the coins are active on coinpaprika.com (please remove inactive ones)

Join logic:

• if joining to base table (i.e. ethereum transactions or traces), I looked to make it an inner join if possible

Incremental logic:

• I used is_incremental & not is_incremental jinja block filters on both base tables and decoded tables
  • where block_time >= date_trunc("day", now() - interval '1 week')
• if joining to base table (i.e. ethereum transactions or traces), I applied join condition where block_time >= date_trunc("day", now() - interval '1 week')
• if joining to prices view, I applied join condition where minute >= date_trunc("day", now() - interval '1 week')

[dune-eng]

Workflow run id 5315198991 approved.

[dune-eng]

Workflow run id 5315198995 approved.

[dune-eng]

Workflow run id 5315409483 approved.

[dune-eng]

Workflow run id 5315409482 approved.

[dune-eng]

Workflow run id 5315608357 approved.

[dune-eng]

Workflow run id 5315608355 approved.

[mike-x7f]

Ethereum trades added to dex_trades_seed and tests have passed.

Test results for other chains (poly, arb, bnb) look good other than missing tokens, which will be added in this PR:
https://dune.com/queries/2648722

Let me know if you have any questions or comments.

[Hosuke]

Hi @mike-x7f,
seems currently this file only contains seed records for the ethereum chain.
Do you want to add xchange seeds for other chains as well?
Like for [bnb, arbitrum, polygon] chains.

[mike-x7f]

I could add some but that test requires both the bought and sold tokens to be in the tokens.erc20 table in order to get the bought and sold amounts. Xchange is a new DEX and doesn't have a lot of volume on those chains yet, and all of the trades so far involve at least one token that isn't in tokens.erc20 yet. I am adding tokens to tokens.erc20 in this PR so I can use check_dex_seed, but I think it will fail until those tokens are added. I ran a manual test on those chains that you can find in the link in my other comment. All of the fields are populated as expected except for those that require tokens.erc20 data (symbol, pair, bought/sold amount).

[jeff-dude]

we can approve with the current state. thanks again @mike-x7f

[mike-x7f]

we can approve with the current state. thanks again @mike-x7f

Thank you Jeff!