Stop using Modis index `rpush:notifications:all` to avoid storage leak by choznerol · Pull Request #743 · rpush/rpush

@choznerol

rpush#525 (comment) mentioned that `rpush:notifications:all` wouldn't be cleaned up properly.

In our case, the `rpush:notifications:all` can keep growing to eat up most of our Redis storage (3.7GB out of 5GB in our case)

If we're not using `Rpush::Client::Redis::Notification.all` anywhere, it might be a good idea to just avoid storing `rpush:notifications:all` at all.

Ref for `enable_all_index`:
- https://github.com/rpush/modis?tab=readme-ov-file#all-index
- rpush/modis#7

@choznerol mentioned this pull request

Dec 23, 2025

@choznerol choznerol changed the title Stop using Modis rpush:notifications:all to avoid storage leak Stop using Modis index rpush:notifications:all to avoid storage leak

Dec 24, 2025