nodejs-storage icon indicating copy to clipboard operation
nodejs-storage copied to clipboard

feat: implement parallel operations

Open ddelgrosso1 opened this issue 3 years ago • 2 comments

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

ddelgrosso1 avatar Sep 15 '22 20:09 ddelgrosso1

Opening as a draft PR while I add in JSDoc comments so that I can start gathering feedback / initial thoughts.

ddelgrosso1 avatar Sep 15 '22 20:09 ddelgrosso1

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.

ddelgrosso1 avatar Sep 16 '22 14:09 ddelgrosso1

Here is the summary of changes.

You are about to add 3 region tags.

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

snippet-bot[bot] avatar Dec 01 '22 20:12 snippet-bot[bot]

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.

ddelgrosso1 avatar Dec 05 '22 16:12 ddelgrosso1