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

Integrates tests for TTL behavior #5045

Open
killme2008 opened this issue Nov 25, 2024 · 0 comments · May be fixed by #5063
Open

Integrates tests for TTL behavior #5045

killme2008 opened this issue Nov 25, 2024 · 0 comments · May be fixed by #5063
Labels
C-enhancement Category Enhancements good first issue Good for newcomers

Comments

@killme2008
Copy link
Contributor

killme2008 commented Nov 25, 2024

What type of enhancement is this?

Tech debt reduction, Other

What does the enhancement do?

GreptimeDB misses integration tests for TTL behavior. The developer only tests it manually or creates some simple cases in unit tests.

We need to add black box tests for it, especially for both of the table engines: the mito engine and the metric engine.

Implementation challenges

Written in sqlness, but needs to trigger the expiration forcibly.

@killme2008 killme2008 added the C-enhancement Category Enhancements label Nov 25, 2024
@killme2008 killme2008 changed the title Integrates tests for TTL behavior. Integrates tests for TTL behavior Nov 25, 2024
@killme2008 killme2008 added the good first issue Good for newcomers label Nov 25, 2024
@killme2008 killme2008 linked a pull request Nov 27, 2024 that will close this issue
3 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-enhancement Category Enhancements good first issue Good for newcomers
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant