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

Increase the default HTTPClient download chunk size to 64 KiB #42896

Merged

Conversation

Calinou
Copy link
Member

@Calinou Calinou commented Oct 19, 2020

This improves download speeds at the cost of increased memory usage.

This change also effects HTTPRequest automatically.

Since this doesn't break backwards compatibility, I marked this as cherry-pickable.

See #32807 and #33862.

@Calinou Calinou requested a review from a team as a code owner October 19, 2020 12:42
@Calinou Calinou added cherrypick:3.x Considered for cherry-picking into a future 3.x release enhancement topic:network labels Oct 19, 2020
@Calinou Calinou added this to the 4.0 milestone Oct 19, 2020
@akien-mga akien-mga requested a review from Faless October 19, 2020 12:53
Copy link
Collaborator

@Faless Faless left a comment

Choose a reason for hiding this comment

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

Looks good. I think it's a good compromise. Thanks 👍

This improves download speeds at the cost of increased memory usage.

This change also effects HTTPRequest automatically.

See godotengine#32807 and godotengine#33862.
@Calinou Calinou force-pushed the httprequest-increase-chunk-size branch from 2b2204e to 1335709 Compare November 7, 2020 15:11
@Calinou Calinou changed the title Increase the default HTTPRequest download chunk size to 64 KiB Increase the default HTTPClient download chunk size to 64 KiB Nov 7, 2020
@akien-mga akien-mga merged commit 7099648 into godotengine:master Nov 7, 2020
@akien-mga
Copy link
Member

Thanks!

@akien-mga
Copy link
Member

Cherry-picked for 3.2.4.

@akien-mga akien-mga removed the cherrypick:3.x Considered for cherry-picking into a future 3.x release label Nov 11, 2020
@Calinou Calinou deleted the httprequest-increase-chunk-size branch March 31, 2021 09:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants