Skip to content

Conversation

@cmraible
Copy link
Collaborator

ref https://linear.app/ghost/issue/NY-865/analytics-sources-not-populating-for-tangle-due-to-408-timeouts

  • The v2 endpoints were created as an experiment to see if the materialized view for mv_session_data would help improve response times and eliminate timeouts at Tinybird endpoints
  • The v2 endpoints backed by the materialized view have a subtle, unintended behavioral change to how visits and pageviews are aggregated and bucketed by time, so we ultimately decided not to promote them to the "default" endpoints
  • This removes the v2 endpoints, the mv_session_data_v2 materialized view and datasource, since they are currently live and consuming server resources, despite not being in active use by any sites.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Dec 17, 2025

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-tinybird-v2-endpoints

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@cmraible
Copy link
Collaborator Author

Need to figure how to actually deploy this change. The Tinybird tests are failing because we can't delete materialized views or datasources without passing a special flag to allow destructive operations. Thinking maybe we can add the flag in CI for this commit, then as soon as the deployment is complete we can go back and remove the flag to avoid accidentally deleting stateful resources in the future.

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