nodejs-storage
nodejs-storage copied to clipboard
feat: implement parallel operations
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:
- [ ] Make sure to open an issue as a bug/issue before writing your code! That way we can discuss the change, evaluate designs, and agree on the general idea
- [ ] Ensure the tests and linter pass
- [ ] Code coverage does not decrease (if any source code was changed)
- [ ] Appropriate docs were updated (if necessary)
Fixes #1868 🦕 Fixes #2051
Opening as a draft PR while I add in JSDoc comments so that I can start gathering feedback / initial thoughts.
Repointing this at a new branch named transfer-manager. My plan is to release this as a "preview" from that branch. Since this is a new feature, keeping it in sync with main should be relatively headache free until such a point that we deem the feature stable.
Here is the summary of changes.
You are about to add 3 region tags.
- samples/downloadFileInChunksWithTransferManager.js:32, tag
storage_download_many_files_transfer_manager - samples/downloadManyFilesWithTransferManager.js:28, tag
storage_download_many_files_transfer_manager - samples/uploadManyFilesWithTransferManager.js:28, tag
storage_upload_many_files_transfer_manager
This comment is generated by snippet-bot.
If you find problems with this result, please file an issue at:
https://github.com/googleapis/repo-automation-bots/issues.
To update this comment, add snippet-bot:force-run label or use the checkbox below:
- [ ] Refresh this comment
Since this is getting large and most feedback has been addressed going to go ahead and merge this into the staging branch. We can re-review before the final merge to main.