Skip to content

Conversation

@dnhatn
Copy link
Member

@dnhatn dnhatn commented Mar 5, 2025

Backports the following commits to 8.x:

This change introduces a cluster setting 
`esql.query.allow_partial_results` that allows enabling or disabling
allow_partial_results in ES|QL at the cluster-wide level. Initially,
this setting defaults to false, but it will be switched to true soon. 
The reason for not changing the default in this PR is that it requires
adjusting many tests, which would make the PR too large. Instead, we
will adjust the tests incrementally and switch the default when the
tests are ready. This cluster setting is useful for falling back to the
previous behavior (i.e., disabling allow_partial_results) if users
upgrade to the new version and haven't updated their queries.

Also, the default setting can be overridden on a per-request basis via a 
URL parameter (allow_partial_results) (changed from request body to URL
parameter to conform to the proposal).

Relates elastic#122802
@dnhatn dnhatn added :Analytics/ES|QL AKA ESQL >enhancement auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) labels Mar 5, 2025
@elasticsearchmachine elasticsearchmachine merged commit e2be510 into elastic:8.x Mar 5, 2025
15 checks passed
@dnhatn dnhatn deleted the backport/8.x/pr-122890 branch March 5, 2025 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >enhancement Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) v8.19.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants