docs: remove threads in transfer manager samples#1029
docs: remove threads in transfer manager samples#1029andrewsg merged 3 commits intogoogleapis:mainfrom
Conversation
|
fyi: |
| # many threads can slow operations, especially with large files, due to | ||
| # contention over the Python GIL. | ||
| # threads=4 | ||
| # The maximum number of worker processes that should be used to handle the |
There was a problem hiding this comment.
Since this sample doesn't mention the worker type constants, let's try omitting them.
"The maximum number of processes to use for the operation. The performance impact of this value depends on the use case, but smaller files usually benefit from a higher number of processes. Each additional process occupies some CPU and memory resources until finished."
There was a problem hiding this comment.
Omitted worker type in comments related to processes. PTAL.
|
@dandhlee Thanks Dan. Since the samples are public-facing I think they could stand to be in the release notes. I'll merge with docs: for now and if the release notes look messy I'll edit them at launch. |
Fixes #1028 🦕