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: tweak transient table data retention settings #15346

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

dantengsky
Copy link
Member

@dantengsky dantengsky commented Apr 26, 2024

I hereby agree to the terms of the CLA available at: https://docs.databend.com/dev/policies/cla/

Summary

Tweak transient table data retention settings

This PR introduces a new setting, transient_data_retention_time_in_minutes, to customize the retention period for transient table. This setting defines how long the historical data should be retained, with a default value of 60 minutes (i.e. 1 hour).

Additionally, when purging data from transient tables, the retention period specified by transient_data_retention_time_in_minutes will now be utilized.

Set transient_data_retention_time_in_minutes to 0 will "restore" the behavior of transient table before this PR.

  • Fixes #[Link the issue here]

Tests

  • Unit Test
  • Logic Test
  • Benchmark Test
  • No Test

Type of change

  • Bug Fix (non-breaking change which fixes an issue)
  • New Feature (non-breaking change which adds functionality)
  • Breaking Change (fix or feature that could cause existing functionality not to work as expected)
  • Documentation Update
  • Refactoring
  • Performance Improvement
  • Other (please describe):

This change is Reviewable

- introduce new setting `transient_data_retention_time_in_minutes`
  which set the retention period (in minutes) of transient table
- during purging data of transient table
  use the value of setting `transient_data_retention_time_in_minutes` in
table navigation
@github-actions github-actions bot added the pr-chore this PR only has small changes that no need to record, like coding styles. label Apr 26, 2024
@dantengsky dantengsky force-pushed the chore-tweak-transient-settings branch from 6aedc87 to 66e66ac Compare April 26, 2024 13:29
@dantengsky dantengsky force-pushed the chore-tweak-transient-settings branch from 66e66ac to 730a84c Compare April 26, 2024 13:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr-chore this PR only has small changes that no need to record, like coding styles.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant