Stop using Modis index `rpush:notifications:all` to avoid storage leak by choznerol · Pull Request #743 · rpush/rpush
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
changed the title
Stop using Modis
Stop using Modis index rpush:notifications:all to avoid storage leakrpush:notifications:all to avoid storage leak
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters