Skip to content
This repository has been archived by the owner on Apr 6, 2023. It is now read-only.

fix(cache): Respect the max_removed_entry configuration #286

Merged
merged 2 commits into from
Jan 5, 2022
Merged

Conversation

ncloudioj
Copy link
Contributor

This fixes #285.

@ncloudioj ncloudioj requested a review from a team as a code owner December 29, 2021 19:24
Copy link
Member

@jrconlin jrconlin left a comment

Choose a reason for hiding this comment

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

looks like more audit flags.

docs/ops.md Outdated
@@ -264,7 +264,7 @@ use.
a matching request is processed.
- `max_removed_entries` - While running the cleanup task, at most this many
entries will be removed before cancelling the task. This should be used to
limit the maximum amount of time the cleanup task takes.
limit the maximum amount of time the cleanup task takes. Defaults to 100000.
Copy link
Member

Choose a reason for hiding this comment

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

nit: consider using the same rust formatting (e.g. 100_000) for reading ease.

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

Successfully merging this pull request may close these issues.

Memory cache doesn't use the max_removed_entries configuration
2 participants