Skip to content

[CI] GeoDistanceIT testDistanceSortingWithUnmappedField failing #126742

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.

Sign up for GitHub

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

Jump to bottom
Closed
elasticsearchmachine opened this issue Apr 12, 2025 · 2 comments · Fixed by #126770
Closed

[CI] GeoDistanceIT testDistanceSortingWithUnmappedField failing #126742

elasticsearchmachine opened this issue Apr 12, 2025 · 2 comments · Fixed by #126770
Assignees
@cbuescher
Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Search Foundations/Search Catch all for Search Foundations Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Apr 12, 2025 •
edited
Loading

Build Scans:

Reproduction Line:

./gradlew ":server:internalClusterTest" --tests "org.elasticsearch.search.sort.GeoDistanceIT.testDistanceSortingWithUnmappedField" -Dtests.seed=6216A97CDE49EF02 -Dtests.locale=nn -Dtests.timezone=America/Creston -Druntime.java=17 -Dtests.fips.enabled=true

Applicable branches:
8.x

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

org.elasticsearch.action.search.SearchPhaseExecutionException:

Issue Reasons:

  • [8.x] 4 failures in test testDistanceSortingWithUnmappedField (1.8% fail rate in 221 executions)
  • [8.x] 2 failures in pipeline elasticsearch-periodic (28.6% fail rate in 7 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Search Foundations/Search Catch all for Search Foundations >test-failure Triaged test failures from CI needs:risk Requires assignment of a risk label (low, medium, blocker) Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch labels Apr 12, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@cbuescher cbuescher self-assigned this Apr 14, 2025
javanna added a commit to javanna/elasticsearch that referenced this issue Apr 14, 2025
@javanna
Remove empty results before merging
e7cf1e9 
We addressed the empty top docs issue with elastic#126385 specifically for scenarios where
empty top docs don't go through the wire. Yet they may be serialized from data node
back to the coord node, in which case they will no longer be equal to Lucene#EMPTY_TOP_DOCS.

This commit expands the existing filtering of empty top docs to include also those that
did go through serialization.

Closes elastic#126742
@javanna javanna mentioned this issue Apr 14, 2025
Merged
@javanna
Copy link
Member

javanna commented Apr 14, 2025

@cbuescher see #126770

javanna added a commit that referenced this issue Apr 17, 2025
@javanna
Remove empty results before merging (#126770)
f274ab7 
We addressed the empty top docs issue with #126385 specifically for scenarios where
empty top docs don't go through the wire. Yet they may be serialized from data node
back to the coord node, in which case they will no longer be equal to Lucene#EMPTY_TOP_DOCS.

This commit expands the existing filtering of empty top docs to include also those that
did go through serialization.

Closes #126742
@javanna javanna mentioned this issue Apr 17, 2025
Merged
elasticsearchmachine pushed a commit that referenced this issue Apr 17, 2025
@javanna
Remove empty results before merging (#126770) (#127031)
21563fa 
We addressed the empty top docs issue with #126385 specifically for scenarios where
empty top docs don't go through the wire. Yet they may be serialized from data node
back to the coord node, in which case they will no longer be equal to Lucene#EMPTY_TOP_DOCS.

This commit expands the existing filtering of empty top docs to include also those that
did go through serialization.

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

@cbuescher cbuescher

Labels
needs:risk Requires assignment of a risk label (low, medium, blocker) :Search Foundations/Search Catch all for Search Foundations Team:Search Foundations Meta label for the Search Foundations team in Elasticsearch >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove empty results before merging javanna/elasticsearch
3 participants
@javanna @cbuescher @elasticsearchmachine