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 global search timeout setting #12211
Conversation
@@ -148,10 +145,11 @@ | |||
private final ParseFieldMatcher parseFieldMatcher; | |||
|
|||
@Inject | |||
public SearchService(Settings settings, ClusterService clusterService, IndicesService indicesService,IndicesWarmer indicesWarmer, ThreadPool threadPool, | |||
public SearchService(Settings settings, SearchSettings searchSettings, ClusterService clusterService, IndicesService indicesService,IndicesWarmer indicesWarmer, ThreadPool threadPool, |
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.
can we just fold SearchSettings
into SearchService
we already have enough classes and abstractions I don't think we need another one.
I left some comments, thanks @jasontedor |
This commit adds a dynamically updatable cluster-level search timeout setting. Closes #12149
@@ -90,7 +91,7 @@ | |||
private ScanContext scanContext; | |||
private float queryBoost = 1.0f; | |||
// timeout in millis | |||
private long timeoutInMillis = -1; | |||
private long timeoutInMillis; |
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.
Can it be final?
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.
oh nevermind :)
LGTM |
Add global search timeout setting
Add documentation for global search timeout Relates #12211
This commit adds a dynamically updatable cluster-level search timeout setting. Additionally, some other search settings were moved to
org.elasticsearch.search.SearchSettings.
Closes #12149