Skip to content

feat(storage): add opt-in raw download support - #9572

Merged
tseaver merged 11 commits into
googleapis:masterfrom
tseaver:9565-storage-add-opt-in-raw-download-support
Nov 5, 2019
Merged

feat(storage): add opt-in raw download support#9572
tseaver merged 11 commits into
googleapis:masterfrom
tseaver:9565-storage-add-opt-in-raw-download-support

Conversation

@tseaver

Copy link
Copy Markdown
Contributor

Note to reviewers: this is likely better viewed commit-by-commit.

Bumps google-resumable-media dependency for google-cloud-storage to require >= 0.5.0, while google-cloud-bigquery just allows it (BigQuery doesn't use the download classes).

Closes#9565.

Avoid depending on implementation details of download objects from
google-resumable-media.
Avoid depending on implementation details of download objects from
google-resumable-media.
Never import module-under-test at module scope of testcase.
Avoid reusing fixtures which do more than needed.
Consolidate duplicated code in prep for adding 'raw_download' option.
If True, uses the 'RawDownload' / 'RawChunkedDownload' helpers from
'google.resumable_media', ensuring that the contents are not expanded
by the requests transport.
If True, uses the 'RawDownload' / 'RawChunkedDownload' helpers from
'google.resumable_media', ensuring that the contents are not expanded
by the requests transport.
@tseavertseaver added the api: storage Issues related to the Cloud Storage API. label Oct 30, 2019
@tseaver
tseaver requested a review from crwilcoxOctober 30, 2019 20:57
@tseaver
tseaver requested a review from frankyn as a code ownerOctober 30, 2019 20:57
@tseaver
tseaver requested a review from a teamOctober 30, 2019 20:57
@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 30, 2019
@tseavertseaver changed the title feat(storage): add opt in raw download supportfeat(storage): add opt-in raw download supportOct 30, 2019

@tswasttswast left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM for BigQuery change

@frankynfrankyn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Will review and marking as requested changes to prevent merges.
Thank you for your patience.

@frankynfrankyn left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks for patience @tseaver.

@tseaver
tseaver merged commit 5f716db into googleapis:masterNov 5, 2019
@tseaver
tseaver deleted the 9565-storage-add-opt-in-raw-download-support branch November 12, 2019 00:16
parthea pushed a commit that referenced this pull request Mar 9, 2026
* deps(bigquery): pin to allow g-r-m 0.5.x
* deps(storage): pin to require g-r-m >= 0.5.0
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storageIssues related to the Cloud Storage API.cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Storage: Add opt-in support for raw downloads.

4 participants

@tseaver@tswast@frankyn@googlebot