Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use custom download chunk size and increase download resume attempts #479

Merged
merged 1 commit into from
Jul 2, 2019

Conversation

tcibinan
Copy link
Contributor

@tcibinan tcibinan commented Jul 2, 2019

Relates to #435 and is intented to improve google storage blobs performance along with #475.

Pull request makes buffering size to be used as download chunk size in order to improve overall google storage blobs download performance.

It was seen that download chunk size increase leads to a higher risk of connection reset. This is way the default download resume attempts were increased from 5 up to 100 to cover most of the possible cases.

Performance benchmarks

Instance type: n1-standard-32
Instance disk: 500 GB
Instance and storage region: europe-west1

Download

File system Size Before, s After, s gsutl
Local 10 GB 74 56 76
Shared 10 GB 92 82 405

Use buffering size as download chunk size in order to improve overall
google storage blobs download performance. Also increase default
download resume attempts to bypass the connection reset issue described
in #435 for most of the possible cases.
@tcibinan tcibinan requested a review from mzueva July 2, 2019 15:28
@mzueva mzueva merged commit 3b9f685 into develop Jul 2, 2019
@tcibinan tcibinan deleted the issue_435-gcp-cli-custom-download-chunk-size branch July 2, 2019 15:36
evgeniimv pushed a commit to evgeniimv/cloud-pipeline that referenced this pull request Oct 14, 2019
…am#479)

Use buffering size as download chunk size in order to improve overall
google storage blobs download performance. Also increase default
download resume attempts to bypass the connection reset issue described
in epam#435 for most of the possible cases.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants