Update go-patricia to 2.2.6 #25809
Merged
Update go-patricia to 2.2.6 #25809
Conversation
This file contains 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
Signed-off-by: Justin Cormack <justin.cormack@docker.com>
|
LGTM 🐸 |
|
LGTM |
liusdu
pushed a commit
to liusdu/moby
that referenced
this pull request
Oct 30, 2017
This patch is squashed by three patches related to go-patricia to fix the memory leak. Docker upstream pr moby#25283 moby#25510 moby#25809 vendor.sh: bump go-patricia to 2.2.4 to fix leaks Signed-off-by: Cristian Staretu <cristian.staretu@gmail.com> (cherry picked from commit 3d714b5) conflicts: hack/vendor.sh Signed-off-by: Liu Hua <sdu.liu@huawei.com> Update go-patricia to 2.2.5 Fixes an issue that showed up on gccgo. Fix moby#25360 Signed-off-by: Justin Cormack <justin.cormack@docker.com> (cherry picked from commit e4402b2) Conflicts: hack/vendor.sh Signed-off-by: Liu Hua <sdu.liu@huawei.com> Update go-patricia to 2.2.6 Signed-off-by: Justin Cormack <justin.cormack@docker.com> (cherry picked from commit d16f962) Conflicts: hack/vendor.sh Signed-off-by: Liu Hua <sdu.liu@huawei.com>
liusdu
pushed a commit
to liusdu/moby
that referenced
this pull request
Oct 30, 2017
Update go patricia this patchset resovled the following issues.： 1. update go-patricia to fix memeory leak * moby#25283 * moby#25510 * moby#25809 2. barkport go-patricia related patch to fix deadlock. * moby#29953 See merge request docker/docker!379
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Minor update, fix possible leak issues.
See #25360 (comment)
Signed-off-by: Justin Cormack justin.cormack@docker.com