Skip to content

Revert "perf(gas_solana_tx_fees): merge compute_limit + compute_unit_price into single compute_budget table"#9896

Merged
0xRobin merged 1 commit into
mainfrom
revert-9880-robin/cur2-2801-perfgas_solana_tx_fees_current-restructure-dual
Jul 17, 2026
Merged

Revert "perf(gas_solana_tx_fees): merge compute_limit + compute_unit_price into single compute_budget table"#9896
0xRobin merged 1 commit into
mainfrom
revert-9880-robin/cur2-2801-perfgas_solana_tx_fees_current-restructure-dual

Conversation

@0xRobin

@0xRobin 0xRobin commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Reverts #9880

@linear-code

linear-code Bot commented Jul 17, 2026

Copy link
Copy Markdown

@cursor

cursor Bot commented Jul 17, 2026

Copy link
Copy Markdown

PR Summary

Medium Risk
Touches fee/prioritization math inputs and doubles instruction_calls scans versus the merged model, but logic matches the prior reverted design with no auth or data-mutation risk.

Overview
Reverts the merged gas_solana_compute_budget approach and restores the pre-#9880 Solana gas fee pipeline.

gas_solana_compute_limit and gas_solana_compute_unit_price are again incremental Delta tables that decode ComputeBudget opcodes 0x02 and 0x03 directly from solana.instruction_calls, instead of thin views over a shared budget table. The combined gas_solana_compute_budget model and its schema/tests are removed.

solana_tx_fees_macro joins those two models separately again (with per-join date predicates) rather than a single compute_budget ref. CI-only 3-day date bounds on transactions, block leaders, prices, and compute budget scans are also dropped from the macro.

Reviewed by Cursor Bugbot for commit f477cf1. Configure here.

@github-actions
github-actions Bot marked this pull request as draft July 17, 2026 14:33
@github-actions github-actions Bot added WIP work in progress dbt: solana covers the Solana dbt subproject labels Jul 17, 2026
@0xRobin
0xRobin marked this pull request as ready for review July 17, 2026 14:33
@github-actions github-actions Bot added ready-for-review this PR development is complete, please review and removed WIP work in progress labels Jul 17, 2026
@0xRobin
0xRobin merged commit 3f9c323 into main Jul 17, 2026
10 of 12 checks passed
@0xRobin
0xRobin deleted the revert-9880-robin/cur2-2801-perfgas_solana_tx_fees_current-restructure-dual branch July 17, 2026 14:34
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 17, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

dbt: solana covers the Solana 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