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)
Mon, Dec 16, 8:19 AM
Unknown Object (File)
Mon, Dec 16, 8:19 AM
Unknown Object (File)
Mon, Dec 16, 8:19 AM
Unknown Object (File)
Mon, Dec 16, 8:19 AM
Unknown Object (File)
Mon, Dec 9, 8:09 AM
Unknown Object (File)
Mon, Nov 25, 6:54 AM
Unknown Object (File)
Sun, Nov 24, 3:49 PM
Unknown Object (File)
Nov 13 2024, 5:28 AM
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
Branch
michal/trans-log-upload
Lint
No Lint Coverage
Unit
No Test Coverage