Increase the default number of threads to 16 #54
Conversation
docs/index.asciidoc
Outdated
| * **Set number of threads correctly.** | ||
|
|
||
| NOTE: The number of threads *must* be greater than or equal to the number of Event hubs plus one. | ||
|
|
There was a problem hiding this comment.
Overall, good information. Thank you!
Unfortunately, we've bumped into some of the formatting limitations (notes in bulleted lists) that we talked about earlier.
To get around these limitations, the content needs to be reworked and restructured. Shall I take a crack at it?
There was a problem hiding this comment.
@karenzone Yes please!
There was a problem hiding this comment.
Hey there. I've come up with something that I think will work. Check this out:
If you like this treatment, we can figure out the best way to incorporate it into your work. For example, if commit it could you cherrypick it into your PR?
023f868
to
3bd6c0a
Compare
|
@robbavey Could you clarify what/where the 16 «standard default» is exactly? |
|
@colinsurprenant When creating an EventProcessorHost, if the Also this comment on an issue requesting information on best practices on the size of the executor service for an EventProcessorHost. |
Increase the default number of threads passed to the EventProcessorHost to 16, to match the standard default.
3bd6c0a
to
9bca3e8
Compare
* master: Bump version to 1.2.1 (logstash-plugins#56) [DOC] Restructure and expand best practices doc (logstash-plugins#55) Increase the default number of threads to 16 (logstash-plugins#54)
Increase the default number of threads passed to the EventProcessorHost
to 16, to match the standard default.