Skip to content

Add TTL eviction to local data cache#16510

Merged
Weiko merged 1 commit intomainfrom
c--add-ttl-eviction-to-local-data-cache
Dec 12, 2025
Merged

Add TTL eviction to local data cache#16510
Weiko merged 1 commit intomainfrom
c--add-ttl-eviction-to-local-data-cache

Conversation

@Weiko
Copy link
Copy Markdown
Member

@Weiko Weiko commented Dec 11, 2025

We keep different versions of our cache to avoid race conditions but we never evict stale data. This PR should fix that

Copy link
Copy Markdown
Contributor

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

No issues found across 3 files

@github-actions
Copy link
Copy Markdown
Contributor

🚀 Preview Environment Ready!

Your preview environment is available at: http://bore.pub:2817

This environment will automatically shut down when the PR is closed or after 5 hours.

Copy link
Copy Markdown
Contributor

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

Your free trial has ended. If you'd like to continue receiving code reviews, you can add a payment method here.

@Weiko Weiko merged commit bd8ed03 into main Dec 12, 2025
56 of 59 checks passed
@Weiko Weiko deleted the c--add-ttl-eviction-to-local-data-cache branch December 12, 2025 09:14
@twenty-eng-sync
Copy link
Copy Markdown

Hey @Weiko! After you've done the QA of your Pull Request, you can mark it as done here. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants