-
Notifications
You must be signed in to change notification settings - Fork 96
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
Fix PB connection pool (pbc_pool_master) leaks #986
Conversation
This also fixes premature pbc checkin to the pool. See also: 60b32b3
push one commit to address @kuenishi 's comment |
Fix in multibag side is needed. So change dep reference to its master branch. |
@@ -54,5 +54,5 @@ | |||
|
|||
{deps_ee, [ | |||
{riak_repl_pb_api,".*",{git,"git@github.com:basho/riak_repl_pb_api.git", {tag, "0.2.5"}}}, | |||
{riak_cs_multibag,".*",{git,"git@github.com:basho/riak_cs_multibag.git", {tag, "1.5.0"}}} | |||
{riak_cs_multibag,".*",{git,"git@github.com:basho/riak_cs_multibag.git", {branch, "master"}}} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would merge the fix in riak_cs_multibag
, create a new tag on that repo, and update this prior to merging. Otherwise you're introducing nondeterminism into the build.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
+1 to Kelly
d9f6fd1
to
20b1265
Compare
20b1265
to
b6ab122
Compare
Fix PB connection pool (pbc_pool_master) leaks Reviewed-by: kuenishi
@borshop merge |
This PR addresses two cases described in #985 .
The specfic requests are in the comment of the above issue.
#985 (comment)