Skip to content

541 drop fee stats agg#2419

Draft
ttboogie wants to merge 3 commits intomainfrom
541-drop-fee-stats-agg
Draft

541 drop fee stats agg#2419
ttboogie wants to merge 3 commits intomainfrom
541-drop-fee-stats-agg

Conversation

@ttboogie
Copy link
Copy Markdown
Contributor

@ttboogie ttboogie commented May 1, 2026

Removes documentation for the deprecated fee_stats_agg model.

Refs stellar/hubble#541

What

  • Deletes docs/data/analytics/hubble/data-catalog/data-dictionary/gold/fee-stats-agg.mdx (the data dictionary page for the legacy model).
  • Updates docs/data/analytics/hubble/analyst-guide/queries-for-horizon-like-data.mdx to drop the fee_stats_agg deprecation note and point readers directly at the two replacement models.

Why

The fee_stats_agg dbt model is being dropped (see stellar/stellar-dbt-public#239) in favor of daily_fee_stats_agg and ledger_fee_stats_agg, which split Classic vs. Soroban transactions and add surge-pricing and inclusion-fee metrics. Both replacements already have data dictionary pages under gold/.

ttboogie and others added 2 commits May 1, 2026 15:22
Removes deprecated fee_stats_agg data dictionary page and updates
the Horizon-like queries guide to reference daily_fee_stats_agg
and ledger_fee_stats_agg as the current fee stats models.

Refs stellar/hubble#541

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Replaces the deprecation note with a single sentence describing
the two replacement models, since fee_stats_agg is being removed.

Refs stellar/hubble#541

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@gre3n3yes34
Copy link
Copy Markdown

gre3n3yes34 commented May 1, 2026 via email

@stellar-jenkins-ci
Copy link
Copy Markdown

CI rebuilds routes.txt and fails on diff. Removing the route
for the deleted fee-stats-agg data dictionary page.

Refs stellar/hubble#541

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

Warning

This change removes routes. The diff of routes is below. Please verify that these changes to the routes are intentional and that redirects are in place if needed. Redirects can be added to nginx/includes/redirects.conf.

diff --git a/routes.txt b/routes.txt
index 0c02d4e5..b4c54887 100644
--- a/routes.txt
+++ b/routes.txt
@@ -225,7 +225,6 @@
 /docs/data/analytics/hubble/data-catalog/data-dictionary/gold
 /docs/data/analytics/hubble/data-catalog/data-dictionary/gold/asset-balances-daily-agg
 /docs/data/analytics/hubble/data-catalog/data-dictionary/gold/daily-fee-stats-agg
-/docs/data/analytics/hubble/data-catalog/data-dictionary/gold/fee-stats-agg
 /docs/data/analytics/hubble/data-catalog/data-dictionary/gold/hourly-fee-agg-account
 /docs/data/analytics/hubble/data-catalog/data-dictionary/gold/hourly-soroban-fee-agg-contract
 /docs/data/analytics/hubble/data-catalog/data-dictionary/gold/ledger-fee-stats-agg

@stellar-jenkins-ci
Copy link
Copy Markdown

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants