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

Pool is not thread safe #20

Closed
purplefox opened this issue Jun 20, 2015 · 2 comments
Closed

Pool is not thread safe #20

purplefox opened this issue Jun 20, 2015 · 2 comments
Assignees

Comments

@purplefox
Copy link
Contributor

@purplefox purplefox commented Jun 20, 2015

I was just looking at the Scala code for the pool, and my knowledge of Scala is poor, but it looks to me that the pool is not thread safe.

@purplefox

This comment has been minimized.

Copy link
Contributor Author

@purplefox purplefox commented Jun 20, 2015

@Narigo

This comment has been minimized.

Copy link
Contributor

@Narigo Narigo commented Jun 21, 2015

Can you share an asynchronous object pool which I could use then? I will most probably get it wrong if it's not working like this. I'd need it for the port to Java anyways then. Sorry for not being able to get this done before 3.0 :(

@cescoffier cescoffier removed the deferred label Jul 7, 2015
@Narigo Narigo mentioned this issue Aug 21, 2015
@cescoffier cescoffier added the Planned label Aug 24, 2015
@cescoffier cescoffier added to review and removed Planned labels Sep 12, 2015
@cescoffier cescoffier closed this Oct 1, 2015
@cescoffier cescoffier removed the to review label Oct 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants
You can’t perform that action at this time.