-
Notifications
You must be signed in to change notification settings - Fork 753
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
feat: new table option data_retention_period_in_hours
#16266
feat: new table option data_retention_period_in_hours
#16266
Conversation
Let's just keep config & table option. I think maybe we don't need the setting. |
do you mean we no longer need session level retention period settings? |
Yes, I think keep three with different priority confuses a lot. |
@soyeric128 This feature need doc. |
I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/
Summary
Introduces new table option
data_retention_period_in_hours
, which has a higher priority to session/global level setting ofdata_retention_period_in_days
, if it is set.The min value of table
data_retention_period_in_hours
is 1 (hour), the max values of it isconfig.data_retention_time_in_days_max
is specified ( or 90 days if not specified in config)Tests
Type of change
This change is