Skip to content

Conversation

@original-brownbear
Copy link
Contributor

Same reasoning as in 2b410c4, with the now weaker ordering guarantees from optimizations this assertion stopped being reliable.

closes #121503

…nTests

Same reasoning as in 2b410c4,
with the now weaker ordering guarantees from optimizations this
assertion stopped being reliable.
@original-brownbear original-brownbear added >test Issues or PRs that are addressing/adding tests :Search Foundations/Search Catch all for Search Foundations v9.0.0 v8.18.0 v9.1.0 labels Feb 7, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-foundations (Team:Search Foundations)

@elasticsearchmachine elasticsearchmachine added the Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch label Feb 7, 2025
@original-brownbear original-brownbear added the auto-backport Automatically create backport pull requests when merged label Feb 13, 2025
@original-brownbear original-brownbear enabled auto-merge (squash) February 13, 2025 15:35
@original-brownbear
Copy link
Contributor Author

Thanks Luca!

@original-brownbear original-brownbear merged commit 5461365 into elastic:main Feb 13, 2025
17 checks passed
@original-brownbear original-brownbear deleted the fix-121503 branch February 13, 2025 16:57
@elasticsearchmachine
Copy link
Collaborator

💔 Backport failed

Status Branch Result
9.0 Commit could not be cherrypicked due to conflicts
8.18 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 122086

original-brownbear added a commit that referenced this pull request Mar 28, 2025
…nTests (#122086) (#125869)

Same reasoning as in 2b410c4,
with the now weaker ordering guarantees from optimizations this
assertion stopped being reliable.

back port of #122086

closes #122911
@javanna
Copy link
Member

javanna commented Mar 31, 2025

@original-brownbear this still needs to be backported to 8.x, correct? Next the backport pending label can be removed?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending :Search Foundations/Search Catch all for Search Foundations Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test Issues or PRs that are addressing/adding tests v8.18.0 v9.0.0 v9.1.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[CI] SearchQueryThenFetchAsyncActionTests testBottomFieldSort failing

3 participants