Page MenuHomePhabricator
Differential D3594

[services] Blob - Fix reading data from S3
ClosedPublic
Actions

Authored by • karol on Apr 1 2022, 3:07 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 8, 2:56 AM
Unknown Object (File)
Wed, Dec 25, 10:20 PM
Unknown Object (File)
Fri, Dec 20, 3:03 PM
Unknown Object (File)
Fri, Dec 20, 3:03 PM
Unknown Object (File)
Fri, Dec 20, 3:03 PM
Unknown Object (File)
Fri, Dec 20, 3:03 PM
Unknown Object (File)
Fri, Dec 20, 3:00 PM
Unknown Object (File)
Sat, Dec 14, 12:09 AM
View All Files
Subscribers
• abosh
• adrian
ashoat
atul
• benschac
tomek

Details

Reviewers
tomek
varun
• max
• jim
Commits
rCOMM656be3bc4746: [services] Blob - Fix reading data from S3
Summary

Depends on D3651

Raised in https://phabricator.ashoat.com/D3470#97072

We want to fix reading the S3 object so it doesn't cut the stream on \n

Test Plan
cd services
yarn test-blob-service

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

• karol created this revision.Apr 1 2022, 3:07 AM
Herald added subscribers: • abosh, • benschac, atul and 3 others. · View Herald TranscriptApr 1 2022, 3:07 AM
• karol mentioned this in D3470: [services] Blob - Add server reactor implementations - get reactor.Apr 1 2022, 3:11 AM
Harbormaster completed remote builds in B7791: Diff 10933.Apr 1 2022, 3:14 AM
• karol requested review of this revision.Apr 1 2022, 3:14 AM
• karol edited the summary of this revision. (Show Details)Apr 1 2022, 3:17 AM
• karol edited the test plan for this revision. (Show Details)
• karol added reviewers: tomek, varun, • max, • jim.
• karol added a parent revision: D3587: [services] Backup - Rename mutex for create new backup.
• karol added a child revision: D3596: [services] Blob - Fix test name for storage manager tests.
tomek accepted this revision.Apr 1 2022, 3:38 AM
tomek added inline comments.
services/blob/src/AwsS3Bucket.cpp
122–123 ↗(On Diff #10933)

Why do we need to resize?

This revision is now accepted and ready to land.Apr 1 2022, 3:38 AM
• karol edited the summary of this revision. (Show Details)Apr 7 2022, 1:20 AM
• karol added a parent revision: D3586: [services] Backup - Send log - add store in database.
• karol edited the summary of this revision. (Show Details)Apr 7 2022, 7:23 AM
• karol added a parent revision: D3651: [services] Backup - Send log - finish read request logic.
• karol added inline comments.Apr 13 2022, 4:53 AM
services/blob/src/AwsS3Bucket.cpp
122–123 ↗(On Diff #10933)

We don't. It is a leftover. It was used to allocate the mem up front.

• karol updated this revision to Diff 11397.Apr 13 2022, 4:55 AM

remove resize

This revision was landed with ongoing or failed builds.Apr 13 2022, 4:58 AM
Closed by commit rCOMM656be3bc4746: [services] Blob - Fix reading data from S3 (authored by • karol). · Explain Why
This revision was automatically updated to reflect the committed changes.
• karol added a commit: rCOMM656be3bc4746: [services] Blob - Fix reading data from S3.
Harbormaster completed remote builds in B8151: Diff 11397.Apr 13 2022, 4:59 AM

Revision Contents
Changeset List

PathSize
services/
blob/
src/
13 lines
Reactors/
server/
7 lines

Diff 11400

View Options

services/blob/src/AwsS3Bucket.cpp

Loading...
View Options

services/blob/src/Reactors/server/GetReactor.h

Loading...