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
Allow to configure custom thread pools #8247
Allow to configure custom thread pools #8247
Conversation
assertThat(info1.getMin(), equalTo(1)); | ||
assertThat(info1.getMax(), equalTo(1)); | ||
assertThat(info1.getQueueSize().singles(), equalTo(1l)); | ||
} |
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.
Should we not fail the test if we get a name thats not either "my_pool1"
or "my_pool2"
?
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.
Make sense. I'll update that.
@martijnvg left a couple of comments |
6fef669
to
1f37ca8
Compare
@colings86 Thanks for looking at this! I updated the PR and applied your feedback. |
LGTM |
1f37ca8
to
7761154
Compare
No description provided.