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

Add alter statements for altering table ttl after table creation #121

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

chhetripradeep
Copy link
Contributor

Signed-off-by: Pradeep Chhetri [email protected]

Tested in local development environment by updating the ttl field in configuration file.

Which problem is this PR solving?

Resolves #120

Short description of the changes

We added alter statements for each of the three tables which gets executed if ttl is greater than 0.
Since materialized views don't support altering ttl configuration, we weren't able to do it for operations table.

Since materialized views don't support altering ttl configuration, we weren't able to do it for operations table.

Signed-off-by: Pradeep Chhetri <[email protected]>
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.

[Feature]: Allow changing TTL configuration on existing tables
1 participant