Page MenuHomePhabricator
Differential D13617

[blob] Move HTTP request/response types to comm-lib
ClosedPublic
Actions

Authored by bartek on Oct 4 2024, 11:43 AM.
Tags
None
Referenced Files
F3335863: D13617.diff
Thu, Nov 21, 11:38 AM
Unknown Object (File)
Fri, Nov 15, 7:05 AM
Unknown Object (File)
Fri, Nov 15, 7:04 AM
Unknown Object (File)
Fri, Nov 15, 2:51 AM
Unknown Object (File)
Thu, Nov 14, 6:48 PM
Unknown Object (File)
Sun, Nov 10, 1:53 PM
Unknown Object (File)
Sat, Nov 9, 11:39 PM
Unknown Object (File)
Thu, Nov 7, 9:09 AM
View All 52 Files
Subscribers
ashoat
tomek

Details

Reviewers
kamil
varun
will
Commits
rCOMMb6f911a681b1: [blob] Move HTTP request/response types to comm-lib
Summary

These types can (and generally should) be shared with the BlobServiceClient living in the comm-lib.
Unified them with types existing in comm-lib so request and response structs are now shared between client and server

Depends on D13616

Test Plan

cargo check

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

bartek created this revision.Oct 4 2024, 11:43 AM
bartek held this revision as a draft.
Herald added subscribers: tomek, ashoat. · View Herald TranscriptOct 4 2024, 11:43 AM
bartek added a child revision: D13618: [blob] Add DB method to query holders by prefix.Oct 4 2024, 11:46 AM
Harbormaster completed remote builds in B32079: Diff 44912.Oct 4 2024, 12:01 PM
bartek retitled this revision from [blob] Move BlobHashAndHolder struct to parent mod to [blob] Move HTTP request/response types to comm-lib.Oct 9 2024, 12:28 AM
bartek edited the summary of this revision. (Show Details)
bartek updated this revision to Diff 44989.Oct 9 2024, 12:31 AM

Share types between client and server

Harbormaster completed remote builds in B32119: Diff 44989.Oct 9 2024, 12:49 AM
bartek published this revision for review.Oct 9 2024, 1:12 AM
bartek added a child revision: D13644: [comm-lib] Add remove_multiple_holders to Blob client.Oct 9 2024, 1:15 AM
bartek mentioned this in D13655: [lib] Add types for Blob Service requests and responses.Oct 9 2024, 2:13 AM
kamil accepted this revision.Oct 10 2024, 1:16 AM
kamil added inline comments.
shared/comm-lib/src/blob/types.rs
1

nit: can you standarize empty lines between structs and comments below? (if there is some pattern I don't see - sorry in advance, but it seems a bit messy)

This revision is now accepted and ready to land.Oct 10 2024, 1:16 AM
Closed by commit rCOMMb6f911a681b1: [blob] Move HTTP request/response types to comm-lib (authored by bartek). · Explain WhyOct 10 2024, 2:06 PM
This revision was automatically updated to reflect the committed changes.
bartek added a commit: rCOMMb6f911a681b1: [blob] Move HTTP request/response types to comm-lib.
bartek mentioned this in rCOMM11b6ebffce87: [lib] Add types for Blob Service requests and responses.

Revision Contents
Changeset List

PathSize
services/
blob/
src/
http/
handlers/
35 lines
69 lines
shared/
comm-lib/
src/
blob/
22 lines
67 lines

Diff 44989

View Options

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

Loading...
View Options

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

Loading...
View Options

shared/comm-lib/src/blob/client.rs

Loading...
View Options

shared/comm-lib/src/blob/types.rs

Loading...