Allow partial results by default in ES|QL - Take 2#127351
Allow partial results by default in ES|QL - Take 2#127351smalyshev merged 10 commits intoelastic:mainfrom
Conversation
…tic#125060)" (elastic#126286)" This reverts commit 8f38b13. Restore changes from elastic#125060 now that the breakage should be fixed.
smalyshev
added
:Analytics/ES|QL
smalyshev
force-pushed
the
enable-partial-results-take-2
branch
from
d2e132d to
7ae2631
Compare
elasticsearchmachine
added
the
Team:Analytics
|
Pinging @elastic/es-analytical-engine (Team:Analytics) |
|
Hi @smalyshev, I've created a changelog YAML for you. |
|
I'll wait with merging it till Monday to avoid disruption if anything goes wrong again. |
|
Do you mind adding a test that checks behavior when we query non-existing index? Or is there one already? |
|
@idegtiarenko There are several that have been added in #126738 and previous fixes. |
💔 Backport failed
You can use sqren/backport to manually backport by running |
|
@smalyshev according to this PR's labels, I need to update the changelog YAML, but I can't because the PR is closed. Please either update the changelog yourself on the appropriate branch, or adjust the labels. Specifically:
|
* Revert "ESQL: Revert "Allow partial results by default in ES|QL (elastic#125060)" (elastic#126286)" This reverts commit 8f38b13. Restore changes from elastic#125060 now that the breakage should be fixed. (cherry picked from commit eb479e5) # Conflicts: # docs/release-notes/breaking-changes.md # x-pack/plugin/build.gradle # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/ClusterComputeHandler.java
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
|
@smalyshev according to this PR's labels, I need to update the changelog YAML, but I can't because the PR is closed. Please either update the changelog yourself on the appropriate branch, or adjust the labels. Specifically:
|
…127474) * Allow partial results by default in ES|QL - Take 2 (#127351) * Revert "ESQL: Revert "Allow partial results by default in ES|QL (#125060)" (#126286)" This reverts commit 8f38b13. Restore changes from #125060 now that the breakage should be fixed. (cherry picked from commit eb479e5) # Conflicts: # docs/release-notes/breaking-changes.md # x-pack/plugin/build.gradle # x-pack/plugin/esql/src/main/java/org/elasticsearch/xpack/esql/plugin/ClusterComputeHandler.java * skip test * fix test * fix test
|
@smalyshev according to this PR's labels, I need to update the changelog YAML, but I can't because the PR is closed. Please either update the changelog yourself on the appropriate branch, or adjust the labels. Specifically:
|
4 participants
This reverts commit 8f38b13.
Restore changes from #125060 now that the breakage should be fixed.
See #125060 for original patch.