[go: up one dir, main page]

Skip to content

MR discussion diffs require compression/decompression when accessing Redis

Everyone can contribute. Help move this issue forward while earning points, leveling up and collecting rewards.

In https://gitlab.com/gitlab-com/gl-infra/scalability/-/issues/418#note_371299552, it was discovered that Projects::MergeRequestsController#discussions was the top Redis cache reader (taking around 45% in the context of the top 3 actions), reading an average of 20gb per day. This was reported in #225600 (closed).

The outcome of #225600 (closed) was to compress the data before it is written to Redis, and then decompress the data on read. While this mitigated the problem of how much data was being persisted in Redis, the longer-term problem of being more efficient with the data (and moving the need for compression) needs to be addressed.

The ~"team::Scalability" label appears on this issue so that we can track the work, but the DRI remains with groupsource code.

Edited by 🤖 GitLab Bot 🤖