Skip to content
This repository was archived by the owner on Oct 13, 2023. It is now read-only.

Conversation

@andrewhsu
Copy link
Contributor

Cherry-pick the commits from:

To fix:

The use of pools.Copy avoids io.Copy's internal buffer allocation.
This commit replaces io.Copy with pools.Copy to avoid the allocation of
buffers in io.Copy.

Signed-off-by: Cristian Staretu <cristian.staretu@gmail.com>
(cherry picked from commit 014095e)
Signed-off-by: Andrew Hsu <andrewhsu@docker.com>
Signed-off-by: Cristian Staretu <cristian.staretu@gmail.com>
(cherry picked from commit ba40f45)
Signed-off-by: Andrew Hsu <andrewhsu@docker.com>
@andrewhsu
Copy link
Contributor Author

LGTM

tests are green

@andrewhsu andrewhsu merged commit b089783 into docker-archive:17.06 Jun 14, 2017
@andrewhsu andrewhsu deleted the pool branch June 14, 2017 18:35
@andrewhsu andrewhsu modified the milestone: 17.06.0 Jul 12, 2017
andrewhsu added a commit that referenced this pull request Apr 26, 2018
[18.05] bump version to 18.05.0-ce-rc1
docker-jenkins pushed a commit that referenced this pull request Oct 9, 2018
Switch copy image to a docker org based one
Upstream-commit: ad08dc12e090bce989ae701ec04f22c26b20d6cf
Component: engine
silvin-lubecki pushed a commit to silvin-lubecki/docker-ce that referenced this pull request Feb 3, 2020
cherry-pick container/stream/attach: use pools.Copy
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants