From 4bab19c126e0ae78bf722538ab83aa8f148f3f3d Mon Sep 17 00:00:00 2001 From: Mouse Reeve Date: Fri, 23 Aug 2024 18:47:31 -0700 Subject: [PATCH] Fixes misunderstanding of how scan_iter yields keys --- bookwyrm/views/admin/redis.py | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/bookwyrm/views/admin/redis.py b/bookwyrm/views/admin/redis.py index 38d390e0f..c682bfcf1 100644 --- a/bookwyrm/views/admin/redis.py +++ b/bookwyrm/views/admin/redis.py @@ -53,11 +53,11 @@ def erase_keys(pattern, count=1000, dry_run=False): """Delete all redis activity keys according to a provided regex pattern""" pipeline = r.pipeline() key_count = 0 - for keys in r.scan_iter(match=pattern, count=count): - key_count += len(keys) - if not dry_run: - for key in keys: - pipeline.delete(key) + for key in r.scan_iter(match=pattern, count=count): + key_count += 1 + if dry_run: + continue + pipeline.delete(key) if not dry_run: pipeline.execute() return key_count