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

Fixed com.netflix.astyanax.connectionpool.exceptions.PoolTimeoutException #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

sbtourist
Copy link

Previously, a com.netflix.astyanax.connectionpool.exceptions.PoolTimeoutException was occurring when running a test with a number of concurrent threads higher than 2.
This was due to MaxBlockedThreadsPerHost not set into com.netflix.jmeter.connections.a6x.AstyanaxConnection.

…ion pool by setting MaxBlockedThreadsPerHost equal to the max number of connections per host.
@sbtourist
Copy link
Author

The problem is, if that configuration isn't set, and you have more threads than configured hosts, exceeding threads will be timed-out by the Astyanax client pool with PoolTimeoutException, making your tests basically fail (try that if you want to be sure, but should be fairly intuitive too).

What you suggest may be correct, but the other way around: that is, set MaxConnsPerHost and MaxBlockedThreadsPerHost equal one to the other by default so that the case above (pretty common with small clusters imho) can work, and eventually override externally.

@sbtourist
Copy link
Author

Any further comments?
Did you have the chance to actually try it?

@epikhinm
Copy link
Contributor

I think this should be another different option with new interface field. Connections not equal threads queue per connection. This option can overflow when i use many connections with many threads.

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

Successfully merging this pull request may close these issues.

None yet

3 participants