Page MenuHomePhabricator
Differential D13482

[blob] Use new blob-exists check in single endpoint
ClosedPublic
Actions

Authored by bartek on Sep 26 2024, 12:40 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Dec 31, 4:29 AM
Unknown Object (File)
Fri, Dec 27, 10:07 AM
Unknown Object (File)
Fri, Dec 27, 7:51 AM
Unknown Object (File)
Fri, Dec 27, 6:26 AM
Unknown Object (File)
Wed, Dec 18, 10:02 AM
Unknown Object (File)
Wed, Dec 18, 10:02 AM
Unknown Object (File)
Wed, Dec 18, 10:02 AM
Unknown Object (File)
Wed, Dec 18, 10:02 AM
View All 40 Files
Subscribers
ashoat
tomek

Details

Reviewers
will
varun
kamil
marcin
Commits
rCOMM8ce683c9558f: [blob] Use new blob-exists check in single endpoint
Summary

Despite D13481, multiple GetItem DDB calls were still happening inside service.assign_holder(). Removed that call there and updated single endpoint to use the new way.

Updated single assign-holder endpoint to use the new batch function.

Depends on D13481

Test Plan
  • Made sure data_exists has still correct value for the POST /blob endpoint
  • Blob integration tests

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
services/
blob/
src/
http/
handlers/
13 lines
5 lines
7 lines

Diff 44587

View Options

services/blob/src/http/handlers/blob.rs

Loading...
View Options

services/blob/src/http/handlers/holders.rs

Loading...
View Options

services/blob/src/service.rs

Loading...