Skip to content
This repository was archived by the owner on Mar 3, 2026. It is now read-only.

samples: add preconditions to storage.objects.rewrite, storage.objects.insert, storage.objects.patch, & storage.objects.delete samples#2046

Merged
shaffeeullah merged 12 commits intomainfrom
shaffeeullah/preconditionsamples
Aug 18, 2022
Merged

samples: add preconditions to storage.objects.rewrite, storage.objects.insert, storage.objects.patch, & storage.objects.delete samples#2046
shaffeeullah merged 12 commits intomainfrom
shaffeeullah/preconditionsamples

Conversation

@shaffeeullah
Copy link
Copy Markdown
Contributor

No description provided.

@shaffeeullah shaffeeullah requested review from a team August 18, 2022 17:29
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. api: storage Issues related to the googleapis/nodejs-storage API. samples Issues that are directly related to samples. labels Aug 18, 2022
shaffeeullah and others added 3 commits August 18, 2022 18:01
* test: add retries (#2039)

* refactor: remove unused `restart` private method (#2038)

Co-authored-by: Sameena Shaffeeullah <shaffeeullah@google.com>

* fix: Retry `EPIPE` Connection Errors + Attempt Retries in Resumable Upload Connection Errors (#2040)

* feat: Add `epipe` as retryable error

* fix: capture and retry potential connection errors

* test: Add tests and remove logs

* test: set `retryOptions` by copy rather than reference

* fix: grammar

* chore(main): release 6.4.1 (#2036)

Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>

* test: add delay to bucket tests to reduce rate limiting errors (#2043)

* samples: storage.objects.insert precondition samples

* updated test

Co-authored-by: Daniel Bankhead <danielbankhead@google.com>
Co-authored-by: release-please[bot] <55107282+release-please[bot]@users.noreply.github.com>
Co-authored-by: Denis DelGrosso <85250797+ddelgrosso1@users.noreply.github.com>
@product-auto-label product-auto-label Bot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels Aug 18, 2022
@product-auto-label product-auto-label Bot added size: m Pull request size is medium. and removed size: l Pull request size is large. labels Aug 18, 2022
…thub.com:googleapis/nodejs-storage into shaffeeullah/preconditionsamples
@shaffeeullah shaffeeullah changed the title samples: add preconditions to storage.objects.rewrite & storage.objects.delete samples samples: add preconditions to storage.objects.rewrite, storage.objects.insert, & storage.objects.delete samples Aug 18, 2022
@product-auto-label product-auto-label Bot added size: l Pull request size is large. and removed size: m Pull request size is medium. labels Aug 18, 2022
@shaffeeullah shaffeeullah changed the title samples: add preconditions to storage.objects.rewrite, storage.objects.insert, & storage.objects.delete samples samples: add preconditions to storage.objects.rewrite, storage.objects.insert, storage.objects.patch, & storage.objects.delete samples Aug 18, 2022
Comment thread samples/composeFile.js Outdated
Comment thread samples/copyFile.js Outdated
Comment thread samples/copyOldVersionOfFile.js Outdated
Copy link
Copy Markdown
Contributor

@ddelgrosso1 ddelgrosso1 left a comment

Choose a reason for hiding this comment

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

lgtm

@shaffeeullah shaffeeullah merged commit ac2de41 into main Aug 18, 2022
@shaffeeullah shaffeeullah deleted the shaffeeullah/preconditionsamples branch August 18, 2022 21:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

api: storage Issues related to the googleapis/nodejs-storage API. samples Issues that are directly related to samples. size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants