Skip to content

chore(dex): decommission 3 unused hidden-leaf spells#9812

Closed
a-monteiro wants to merge 1 commit into
mainfrom
andre/cur2-2855-dex-decommission
Closed

chore(dex): decommission 3 unused hidden-leaf spells#9812
a-monteiro wants to merge 1 commit into
mainfrom
andre/cur2-2855-dex-decommission

Conversation

@a-monteiro

@a-monteiro a-monteiro commented Jun 20, 2026

Copy link
Copy Markdown
Member

Decommissions 3 unused hidden-leaf spells in dex as part of the compute-optimization quick wins.

Each spell is hidden (hide_spells() -> dune.visible=false), a leaf (no other dbt/sqlmesh model, exposure, or curated-data reference -- only its own schema tests; the dex.trades union consumes *_base_trades, not these per-project .trades views), and unused by customers (zero enterprise/paid queries over Jan-Jun 2026 in QES, below the negligible-free bar). Removed: paraswap_delta_v2_optimism.trades, gamma.uniswap_pools, odos_avalanche_c.trades.

Compute here is marginal (~0.03 CPU-hrs/day combined) -- this batch is hygiene/completeness; the real savings are in the daily and hourly batches.

The 4th candidate from CUR2-2855, zeroex_polygon.nft_fills (0.000 CPU-hrs/day), is deferred: its schema block is the anchor-definition source for the rest of zeroex_polygon_schema.yml, so removing it requires relocating 14 YAML anchors into surviving sibling models. Not worth the structural churn for zero compute savings; can be done separately if desired.

dbt parse is clean; surviving sibling models in the edited paraswap schema are byte-identical.

Towards CUR2-2855
Towards CUR2-2852

@github-actions github-actions Bot added WIP work in progress dbt: dex covers the DEX dbt subproject labels Jun 20, 2026

Copy link
Copy Markdown
Member Author

This stack of pull requests is managed by Graphite. Learn more about stacking.

@a-monteiro a-monteiro marked this pull request as ready for review June 20, 2026 21:40
@cursor

cursor Bot commented Jun 20, 2026

Copy link
Copy Markdown

PR Summary

Low Risk
Deletes unused hidden models with no remaining dbt references; downstream dex.trades paths use other models, so impact is limited to stopping marginal scheduled builds.

Overview
Removes three hidden-leaf dbt spells from the dex subproject as compute/hygiene cleanup: the gamma.uniswap_pools union over Optimism Gamma pools, odos_avalanche_c.trades (legacy Spark aggregator spell marked prod_exclude), and paraswap_delta_v2_optimism.trades (thin wrapper around delta_v2_master).

Each deletion includes the matching _schema.yml entries (tests and column docs). Chain-specific models such as gamma_optimism_uniswap_pools and other Paraswap Optimism models are unchanged; only these unused surfaced tables go away.

Reviewed by Cursor Bugbot for commit 6d93b6b. Configure here.

@github-actions github-actions Bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jun 20, 2026
@a-monteiro a-monteiro requested a review from a team June 22, 2026 07:11
@0xRobin 0xRobin self-assigned this Jun 22, 2026
@0xRobin 0xRobin closed this Jun 29, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 29, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dbt: dex covers the DEX dbt subproject ready-for-review this PR development is complete, please review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants