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

Add node-level field caps requests #79214

Merged
merged 2 commits into from
Oct 15, 2021
Merged

Conversation

dnhatn
Copy link
Member

@dnhatn dnhatn commented Oct 15, 2021

This PR was reviewed already.

Currently to gather field caps, the coordinator sends a separate transport
request per index. When the original request targets many indices, the overhead
of all these sub-requests can add up and hurt performance. This PR switches the
execution strategy to reduce the number of transport requests: it groups
together the index requests that target the same node, then sends only one
request to each node.

Co-authored-by: Julie Tibshirani [email protected]

Relates #77047
Relates # #78647

jtibshirani and others added 2 commits October 14, 2021 23:01
Currently to gather field caps, the coordinator sends a separate transport
request per index. When the original request targets many indices, the overhead
of all these sub-requests can add up and hurt performance. This PR switches the
execution strategy to reduce the number of transport requests: it groups
together the index requests that target the same node, then sends only one
request to each node.
This adds a retry mechanism for node level field caps requests 
introduced in elastic#77047.
@dnhatn dnhatn added backport and removed v7.16.0 labels Oct 15, 2021
@dnhatn dnhatn marked this pull request as ready for review October 15, 2021 22:37
@dnhatn dnhatn merged commit 60dba05 into elastic:7.x Oct 15, 2021
@dnhatn dnhatn deleted the 7x-backport-field-caps branch October 15, 2021 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants