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.

Sign up for GitHub

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

Jump to bottom

fix: cache should be stored in a hour #13

Merged
merged 3 commits into from
Dec 24, 2022
Merged

fix: cache should be stored in a hour #13

merged 3 commits into from
Dec 24, 2022

Conversation

soujiokita
Copy link
Contributor

CC: Redis Email Alert

Hello Okita Alter,

The total size of datasets under your plan has reached 81.66% of the memory limit

Free tier of redis only 30MB, the default configuration should follow this behaviour or it's mess

@da-vinci-bot
Copy link

da-vinci-bot bot commented Dec 24, 2022

Hey?! @soujiokita Thanks for making a pull request, an collaborators will get back to you soon! UwU


Click here to make your day UmU
abc

@soujiokita
Copy link
Contributor Author

All tests passed? except that i don't have any permissions to publish on your base repo and docker

sinkaroid
sinkaroid approved these changes Dec 24, 2022
View reviewed changes
Copy link
Owner

@sinkaroid sinkaroid left a comment

Choose a reason for hiding this comment

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

Oh i see, thanks

@sinkaroid sinkaroid merged commit c316ae5 into sinkaroid:master Dec 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sinkaroid sinkaroid sinkaroid approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@soujiokita @sinkaroid