gitaly: Delete log entry after being applied (!5341) · Merge requests · GitLab.org / gitaly

Karthik Nayak requested to merge 4735-prune-write-ahead-log-entries-after-application into master Feb 03, 2023

After a log entry has been applied to the repository, we should ideally delete that entry from the write-ahead log (in our case, the badger database).

Historically we didn't do this, because keeping the entries made it easier to verify database state in the tests.

In this MR we first add mocks over the badger database which also tracks the writes made to the database. Then replace the tests to use the mocks. Finally we delete the log entries after being applied.

Closes #4735 (closed)

gitaly: Delete log entry after being applied

Merge request reports