Page MenuHomePhabricator

[services] Backup - Rename mutex for create new backup
AbandonedPublic

Authored by karol on Mar 31 2022, 6:32 AM.
Tags
None
Referenced Files
F3376867: D3587.id10883.diff
Wed, Nov 27, 2:40 AM
Unknown Object (File)
Fri, Nov 22, 10:55 PM
Unknown Object (File)
Wed, Nov 20, 4:17 AM
Unknown Object (File)
Fri, Nov 15, 11:26 PM
Unknown Object (File)
Tue, Nov 5, 5:47 PM
Unknown Object (File)
Sat, Nov 2, 4:00 PM
Unknown Object (File)
Oct 13 2024, 4:00 PM
Unknown Object (File)
Oct 13 2024, 3:59 PM

Details

Summary

Depends on D3586

Rename mutex in create new backup reactor to be consistent with the send log reactor.

Test Plan
cd services
yarn run-backup-service

Diff Detail

Repository
rCOMM Comm
Branch
arcpatch-D3488
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

karol edited the test plan for this revision. (Show Details)
karol added reviewers: max, jim, varun, tomek.
tomek requested changes to this revision.Apr 7 2022, 12:04 AM

It would be a lot better to modify the original diff instead of adding a followup like this. The rename would significantly improve the readability and this is likely that it will be requested in the original diff.

This revision now requires changes to proceed.Apr 7 2022, 12:04 AM

changed in the original diff