You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
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.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Labels
dbt: dexcovers the DEX dbt subprojectready-for-reviewthis PR development is complete, please review
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Decommissions 3 unused hidden-leaf spells in
dexas 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; thedex.tradesunion consumes*_base_trades, not these per-project.tradesviews), 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 ofzeroex_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 parseis clean; surviving sibling models in the edited paraswap schema are byte-identical.Towards CUR2-2855
Towards CUR2-2852