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

Add mod status in comment.list response #158

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

keikari
Copy link
Contributor

@keikari keikari commented Aug 12, 2024

No description provided.

@tzarebczan
Copy link
Contributor

do we need any db caching for this, or we think it will be performant enough?

@nikooo777
Copy link
Member

So, I took a deeper look at this, it's problematic because it doesn't use a cache, which would cause quite a load on the database.
If you're able to add that, and examples of ccache usage are right around there, then we can merge it in

@keikari
Copy link
Contributor Author

keikari commented Oct 9, 2024

Wasn't sure what numbers use to the ccache. Current setting maxItems: 100000 and expirationTime: 10mins

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants