[blob] Make revoke holder transaction retryable
ClosedPublic
Actions

Authored by bartek on Feb 14 2025, 12:20 PM.

Details

Reviewers

kamil
tomek

Commits

rCOMM178a0cfc022f: [blob] Make revoke holder transaction retryable

Summary

ENG-10134.
Simultaneous removal of two holders for the same blob might result in TransactionConflict.
Added exponential backoff for retries.

Depends on D14363

Test Plan

Created a blob with 10 holders.
Used a script to start 10 concurrent holder removal requests.
Some of them failed before, afterwards they all succeed.

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Feb 14 2025, 12:20 PM

bartek held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFeb 14 2025, 12:20 PM

Harbormaster completed remote builds in B33515: Diff 47105.Feb 14 2025, 12:39 PM

bartek published this revision for review.Feb 14 2025, 3:15 PM

kamil accepted this revision.Feb 17 2025, 10:54 AM

This revision is now accepted and ready to land.Feb 17 2025, 10:54 AM

Closed by commit rCOMM178a0cfc022f: [blob] Make revoke holder transaction retryable. · Explain WhyFeb 17 2025, 11:31 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMM178a0cfc022f: [blob] Make revoke holder transaction retryable.

Revision Contents
Changeset List

Path

Size

services/

blob/

src/

database/

client.rs

34 lines

Diff 47105

View Options

services/blob/src/database/client.rs

[blob] Make revoke holder transaction retryableClosedPublicActions