FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

feat: Transfer Manager ParallelCompositeUploads by sydney-munro · Pull Request #2494 · googleapis/java-storage · GitHub

feat: Transfer Manager ParallelCompositeUploads - #2494

Merged
sydney-munro merged 10 commits into
mainfrom
tm-pcu
Apr 11, 2024
Merged

feat: Transfer Manager ParallelCompositeUploads#2494
sydney-munro merged 10 commits into
mainfrom
tm-pcu

Conversation

sydney-munro commented Apr 10, 2024
edited
Loading

Copy link
Copy Markdown
Contributor

This implements ParallelCompositeUploads(PCU) in Transfer Manager.

ParallelCompositeUpload in Transfer Manager is an opt-in feature, customers will have to update their config to allow Transfer Manager to perform PCU operations on their behalf. Transfer Manager will only do this if the following conditions are met:

  • The customer has set allowParallelCompositeUpload to true
  • The file to be uploaded is 4 * the buffer per worker size
  • The customer has specified enough threads that ParallelCompositeUpload can be of benefit while still cuing other work.

product-auto-label Bot added size: l Pull request size is large. api: storage Issues related to the googleapis/java-storage API. labels Apr 10, 2024
sydney-munro marked this pull request as ready for review April 11, 2024 19:24
sydney-munro requested a review from a team April 11, 2024 19:24
sydney-munro requested a review from a team April 11, 2024 20:31
sydney-munro added the owlbot:ignore instruct owl-bot to ignore a PR label Apr 11, 2024

BenWhitehead left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

One cleanup item related to options.

Can you also purge out the owlbot commit, it's adding noise to the PR unrelated to your change.

sydney-munro merged commit 8b54549 into main Apr 11, 2024
sydney-munro deleted the tm-pcu branch April 11, 2024 22:43
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the googleapis/java-storage API. owlbot:ignore instruct owl-bot to ignore a PR size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants


Back | FazBrowse Home | New Git URL