added executor_threads documentation#421
Conversation
|
Here was a parallel one: |
andsel
left a comment
There was a problem hiding this comment.
Just left a minimal suggestion
|
that description is more extended, we could use that, and close that draft PR with this one |
|
Hi @111andre111 I've rebased to |
|
Yes correct @111andre111, but Changelog needs a fix |
|
@andsel I have fixed now the Changelog |
karenzone
left a comment
There was a problem hiding this comment.
Thank you for adding this important documentation. Nice job explaining it. I left some minor suggestions inline for your consideration.
andsel
left a comment
There was a problem hiding this comment.
LGTM, merge squashing the commits please, or ping me if you need I do it
|
@andsel Sorry if I didn't manage the squash. I always got an error message. |
|
@111andre111 I've rebased to |
|
Ah I see. I didn't see that. Thank you @andsel |
Thanks for contributing to Logstash! If you haven't already signed our CLA, here's a handy link: https://www.elastic.co/contributor-agreement/