Skip to content

Conversation

@JVerwolf
Copy link
Contributor

@JVerwolf JVerwolf commented Jul 8, 2025

WIP

jdconrad and others added 21 commits July 14, 2025 16:28
…tic#131235)

Some tests were seeing an unexpected warning header from wiping all
indices in the test cleanup, caused by the `.triggered-watches` data
stream being present. We exclude the backing indices of this data stream
from the indices wipe to avoid the warning header. The presence of this
data stream shouldn't intefere with tests, and any tests that do
interact with that data stream should assert/control its state by
themselves to ensure consistency.

Fixes elastic#129682
Replaces the deprecated method with appropriate substitute.
Currently, errors from the field-caps phase are not always handled
properly, leading to cases where the final response is not marked as
partial correctly. For example: FROM ok*,unavailable_index* should
return a partial result, as unavailable_index* is skipped after the
resolution phase. This change tracks failures that occur during
field-caps and reports them in the final response. Since this only
affects cases with allow_partial_results=true, I am labeling this as a
non-issue and will backport the change to 9.1 and 8.19.
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.

5 participants