Skip to content
This repository was archived by the owner on Dec 3, 2019. It is now read-only.

partitioned db pool #88

Merged
merged 1 commit into from
Jun 2, 2014
Merged

partitioned db pool #88

merged 1 commit into from
Jun 2, 2014

Conversation

fwbrasil
Copy link
Collaborator

The single threaded approach became a bottleneck for me.

fixes #58

@mauricio
Copy link
Owner

@fwbrasil could you get some docs on how it works and why someone should pick this one?

@fwbrasil
Copy link
Collaborator Author

@mauricio I was thinking about this. Maybe we should not allow or discourage the usage of the SingleThreaded pool. The partitioned pool overhead is low and if necessary, it is possible to use it with partitions = 1. Wdyt?

@mauricio
Copy link
Owner

That makes sense. I'm going to do the other async stuff this week and we
can deprecate this.

Em segunda-feira, 12 de maio de 2014, Flavio W. Brasil <
notifications@github.com> escreveu:

@mauricio https://github.com/mauricio I was thinking about this. Maybe
we should not allow or discourage the usage of the SingleThreaded pool. The
partitioned pool overhead is low and if necessary, it is possible to use it
with partitions = 1. Wdyt?

Reply to this email directly or view it on GitHubhttps://github.com//pull/88#issuecomment-42809175
.

Maurício Linhares
http://mauricio.github.io/ - http://twitter.com/#!/mauriciojr

@fwbrasil
Copy link
Collaborator Author

fwbrasil commented Jun 2, 2014

@mauricio can we merge this pr?

mauricio added a commit that referenced this pull request Jun 2, 2014
@mauricio mauricio merged commit 6db0d9c into mauricio:master Jun 2, 2014
@mauricio
Copy link
Owner

mauricio commented Jun 2, 2014

@fwbrasil can you include scaladoc for the class itself explaining it's usage?

@fwbrasil
Copy link
Collaborator Author

fwbrasil commented Jun 2, 2014

@mauricio Sorry, I forgot about the scaladoc, will provide it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

SingleThreadedAsyncObjectPool alternatives
2 participants