Skip to content

Updating bulk get sample for multi-blob download and improved clarity #36

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
RachelTucker merged 2 commits into from
Apr 9, 2020
Merged

Updating bulk get sample for multi-blob download and improved clarity #36

RachelTucker merged 2 commits into from
Apr 9, 2020

Conversation

@RachelTucker
Copy link
Contributor

@RachelTucker RachelTucker commented Apr 2, 2020

  • The previous sample did not work with blobbed objects. New one does.
  • Reworked sample to make temporary file management more obvious. This makes it more clear how to transform the sample code into production code.
  • Added more explanation around each step to make it more clear for someone who is new to the BP workflow.

@RachelTucker RachelTucker requested a review from rpmoore April 2, 2020 23:10
@RachelTucker RachelTucker merged commit 054e4b2 into SpectraLogic:master Apr 9, 2020
@RachelTucker RachelTucker deleted the updating_samples branch April 9, 2020 21:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@rpmoore rpmoore Awaiting requested review from rpmoore

1 more reviewer

@shabtaisharon shabtaisharon shabtaisharon approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@RachelTucker @shabtaisharon