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

Jump to bottom

Access to pool from Wrapper #113

Closed
ook opened this issue Feb 28, 2019 · 1 comment
Closed

Access to pool from Wrapper #113

ook opened this issue Feb 28, 2019 · 1 comment

Comments

@ook
Copy link
Contributor

ook commented Feb 28, 2019

When using ConnectionPool::Wrapper, we can't access the @pool definition, so it prevents us from checking the pool status.

Let's give that access!
ook added a commit to ook/connection_pool that referenced this issue Feb 28, 2019
@ook
mperham#113 expose pool from Wrapper
3e92a02
@ook
Copy link
Contributor Author

ook commented Feb 28, 2019

(PR opened)

ook added a commit to ook/connection_pool that referenced this issue Feb 28, 2019
@ook
mperham#113 Fill Changes.md
1969c61
@ook ook mentioned this issue Feb 28, 2019
Merged
@mperham mperham closed this as completed Feb 28, 2019
ook added a commit to ook/connection_pool that referenced this issue Mar 1, 2019
@ook
mperham#113 PR comment: rename #pool to #wrapped_pool
03c3f56
mperham pushed a commit that referenced this issue Mar 1, 2019
@ook @mperham
#113 expose pool from Wrapper (#114)
99b81aa 
* #113 expose pool from Wrapper

* #113 Fill Changes.md

* #113 PR comment: rename #pool to #wrapped_pool
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@ook @mperham