Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: explain performance improvements. #3538

Conversation

billy-the-fish
Copy link
Contributor

Enable CAGGS to use MERGE for refresh operations rather than delete

Copy link

Allow 10 minutes from last push for the staging site to build. If the link doesn't work, try using incognito mode instead. For internal reviewers, check web-documentation repo actions for staging build status. Link to build for this PR: http://docs-dev.timescale.com/docs-3513-docs-rfc-add-documentation-about-using-merge-for-continuous-aggregate-refresh

Copy link
Contributor

@iroussos iroussos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good to me @billy-the-fish, thank you for adding this to the docs!

Two comments:

(1) This is a parameter that you set and forget when you create continuous aggregates, not something that you need to set when you manually refresh a specific continuous aggregate.

Feels to me that this should be part of the CREATE MATERIALIZED VIEW (Continuous Aggregate) page. What do you think?

(2) A small nitpick:
The "Configure database parameters" page only focuses on the Timescale Cloud UI, not mentioning how a self hosted user running TimescaleDB can enable those parameters. But this is a core feature of TimescaleDB available to all users.

Should we either add the command here or add a section on that page on how to enable parameters through psql?

@billy-the-fish
Copy link
Contributor Author

Moved the config param to CREATE MATERIALIZED VIEW (Continuous Aggregate) and linked to it from the refresh page, added the config to the TimescaleDB ref.

@billy-the-fish billy-the-fish merged commit 2f4f27b into latest Oct 31, 2024
3 checks passed
@billy-the-fish billy-the-fish deleted the 3513-docs-rfc-add-documentation-about-using-merge-for-continuous-aggregate-refresh branch October 31, 2024 13:14
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.

[Docs RFC] Add documentation about using MERGE for Continuous Aggregate refresh
2 participants