-
Notifications
You must be signed in to change notification settings - Fork 24.9k
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
DOCS: Make ellipsis optional in /cat/thread_pool #33186
Conversation
The fix proposed in elastic#31442 fails with the oss distro because the added 3dots does not match anything with the default oss while a 3dots expression requires matching at least one thread pool. This change adjusts the thread pool list so that it can match both the oss (without ccr) and default distro (with ccr).
Pinging @elastic/es-distributed |
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.
I left a comment.
@@ -16,10 +16,11 @@ Which looks like: | |||
-------------------------------------------------- | |||
node-0 analyze 0 0 0 | |||
... |
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.
Hmm...I was thinking instead we should make the ...
translate to an optional pattern that would match the specified thread pool format (node-\d+ \S+ \d+ \d+ \d+). That way we do not have to require there be a thread pool between analyze and fetch_shard_started.
This reverts commit dca38d6.
@jasontedor Yep, I've made 3dots optional! |
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.
LGTM!
Thanks @jasontedor for reviewing! |
The fix proposed in #31442 fails with the oss distro because the added 3dots does not match anything with the default oss while a 3dots expression requires matching at least one thread pool. This change makes an ellipsis optional so the thread_pool list can match both the oss distro (without ccr) and default distro (with ccr). Relates #31442
* ccr: DOCS: Make ellipsis optional in /cat/thread_pool (elastic#33186)
The fix proposed in #31442 fails with the oss distro because the added
3dots does not match anything with the default oss while a 3dots
expression requires matching at least one thread pool.
This change makes a 3dots optional so the thread_pool list can match
both the oss distro (without ccr) and default distro (with ccr).
Relates #31442