Page MenuHomePhabricator

[backup] Revoke holders of replaced log items
ClosedPublic

Authored by michal on Mar 19 2024, 10:33 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 26, 12:57 AM
Unknown Object (File)
Thu, Apr 25, 12:49 PM
Unknown Object (File)
Thu, Apr 25, 12:48 PM
Unknown Object (File)
Mon, Apr 15, 11:57 PM
Unknown Object (File)
Sat, Apr 6, 12:14 AM
Unknown Object (File)
Tue, Apr 2, 9:16 AM
Unknown Object (File)
Mon, Apr 1, 3:21 AM
Unknown Object (File)
Sun, Mar 31, 6:56 PM
Subscribers

Details

Summary

ENG-7005 : Handle blob holders of replaced logs
It's possible that backup logs will be replaced if there is an upload request send with the same log id (and we need to allow it as a part of the transactional log upload). This diff fixes an issue where we wouldn't clear the holders of the replaced logs.

Test Plan

Modify the backup integration test to only send logs with id of 1. Made sure that at the end there were only two blobs with holders -> one for the compaction and one for the last log sent.

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable