[identity] Send RETRY for transaction conflicts
AcceptedPublic
Actions

Authored by bartek on Wed, Oct 2, 1:07 AM.

Details

Reviewers

varun
will

Summary

Part of ENG-9427.

Added a function that checks if DDB error is transaction conflict (we already have one but this one is more opinionated)
Added a case to handle_db_error to send retry tonic message instead of unexpected error

Depends on D13567

Test Plan

Simulated TransactionConflictException DDB error and checked gRPC status code in a UI client

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Wed, Oct 2, 1:07 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptWed, Oct 2, 1:07 AM

bartek added a child revision: D13569: [identity] Add exponential backoff to refresh_user_prekeys.Wed, Oct 2, 1:10 AM

Harbormaster completed remote builds in B32008: Diff 44812.Wed, Oct 2, 1:24 AM

bartek published this revision for review.Wed, Oct 2, 1:26 AM

will accepted this revision.Wed, Oct 2, 6:07 AM

This revision is now accepted and ready to land.Wed, Oct 2, 6:07 AM

Revision Contents
Changeset List

Path

Size

services/

identity/

src/

client_service.rs

6 lines

ddb_utils.rs

24 lines

Diff 44812

View Options

services/identity/src/client_service.rs