Page MenuHomePhabricator
Differential D8974

[services-lib][blob-client] Use Bytes instead of Vec<u8>
ClosedPublic
Actions

Authored by bartek on Aug 28 2023, 4:06 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Dec 14, 7:42 PM
Unknown Object (File)
Sat, Dec 14, 7:42 PM
Unknown Object (File)
Sat, Dec 14, 7:41 PM
Unknown Object (File)
Sat, Dec 14, 7:30 PM
Unknown Object (File)
Sun, Dec 8, 2:18 PM
Unknown Object (File)
Mon, Nov 25, 8:05 PM
Unknown Object (File)
Nov 22 2024, 10:46 AM
Unknown Object (File)
Nov 22 2024, 4:52 AM
View All 84 Files
Subscribers
ashoat
tomek

Details

Reviewers
michal
• jon
Commits
rCOMM758d5a4d6450: [services-lib][blob-client] Use Bytes instead of Vec<u8>
Summary

Partially overlaps with D8964 - we should use Bytes instead of Vec<u8> for blob data to avoid unnecessary allocations.

Also added the bytes dependency to services-lib and re-exported it.

Depends on D8786

Test Plan

Tested together with the next diff

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Revision Contents
Changeset List

PathSize
services/
backup/
comm-services-lib/
8 lines
src/
blob/
14 lines
9 lines
reports/

Diff 30519

View Options

services/backup/Cargo.lock

Loading...
View Options

services/comm-services-lib/Cargo.lock

Loading...
View Options

services/comm-services-lib/Cargo.toml

Loading...
View Options

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

Loading...
View Options

services/comm-services-lib/src/lib.rs

Loading...
View Options

services/reports/Cargo.lock

Loading...