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

BTE occasionally fails to set TTL for cached qEdges #757

Closed
tokebe opened this issue Oct 31, 2023 · 2 comments
Closed

BTE occasionally fails to set TTL for cached qEdges #757

tokebe opened this issue Oct 31, 2023 · 2 comments
Assignees
Labels
bug Something isn't working On Test Related changes are deployed to Test server

Comments

@tokebe
Copy link
Member

tokebe commented Oct 31, 2023

It's been demonstrated that BTE will occasionally "hold onto" some cached qEdges beyond their expected expire time. The most likely explanation for this is that BTE is failing to set an expire time, likely by way of being restarted prior to the expire time being set.

There is a likely better way to handle this, but as redis does not store key age, the simplest naive approach is to just add a cron job that eliminates all cached qEdges after some interval (once a day at midnight?).

@tokebe tokebe self-assigned this Oct 31, 2023
@tokebe tokebe added the bug Something isn't working label Oct 31, 2023
@colleenXu
Copy link
Collaborator

colleenXu commented Nov 28, 2023

I wonder if this is related to #761

EDIT: also linking to our old Slack discussion on this topic and the slack post where @tokebe tried to fix this

@tokebe tokebe added the On Dev Related changes are deployed to Dev server label Dec 11, 2023
@tokebe tokebe added On CI Related changes are deployed to CI server On Test Related changes are deployed to Test server and removed On Dev Related changes are deployed to Dev server On CI Related changes are deployed to CI server labels Dec 18, 2023
@tokebe
Copy link
Member Author

tokebe commented Feb 21, 2024

Relevant changes deployed to Prod.

@tokebe tokebe closed this as completed Feb 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working On Test Related changes are deployed to Test server
Projects
None yet
Development

No branches or pull requests

2 participants