Skip to content
This repository has been archived by the owner on Dec 14, 2018. It is now read-only.

Concurrent connection on redis cache leads to connection leak #257

Closed
JunTaoLuo opened this issue Dec 16, 2016 · 5 comments
Closed

Concurrent connection on redis cache leads to connection leak #257

JunTaoLuo opened this issue Dec 16, 2016 · 5 comments
Assignees
@JunTaoLuo
Labels
3 - Done bug patch-approved
Milestone
1.0.2

Comments

@JunTaoLuo
Copy link
Contributor

Need to port fix in 3522082 to 1.0.4 branch when available.
@JunTaoLuo JunTaoLuo added this to the 1.0.4 milestone Dec 16, 2016
@muratg muratg mentioned this issue Jan 11, 2017
Closed
@muratg muratg modified the milestones: 1.0.2, 1.0.4 Jan 11, 2017
@muratg
Copy link

muratg commented Jan 11, 2017

(Setting milestone to 1.0.2 to reflect the actual package version to be shipped.)

@Eilon
Copy link
Member

Eilon commented Jan 19, 2017

This patch bug is approved. Please use the normal code review process w/ a PR and make sure the fix is in the correct branch, then close the bug and mark it as done.

@Eilon
Copy link
Member

Eilon commented Jan 19, 2017

@muratg please assign this bug.

@JunTaoLuo JunTaoLuo self-assigned this Jan 19, 2017
This was referenced Jan 21, 2017
Merged
Merged
@JunTaoLuo
Copy link
Contributor Author

Patch merged. Leaving the issue open since version numbers will need to be updated.

@JunTaoLuo
Copy link
Contributor Author

Version numbers updated

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@JunTaoLuo JunTaoLuo

Labels
3 - Done bug patch-approved
Projects
None yet
Milestone
1.0.2
Development

No branches or pull requests
3 participants
@Eilon @muratg @JunTaoLuo