Skip to content
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

[CI] AbstractShapeGeometryFieldMapperTests testCartesianBoundsBlockLoader failing #119201

Open
elasticsearchmachine opened this issue Dec 22, 2024 · 4 comments · Fixed by #119265
Open

[CI] AbstractShapeGeometryFieldMapperTests testCartesianBoundsBlockLoader failing #119201

elasticsearchmachine opened this issue Dec 22, 2024 · 4 comments · Fixed by #119265
Assignees
@GalLalouche
Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes low-risk An open issue or test failure that is a low risk to future releases Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI

Comments

@elasticsearchmachine
Copy link
Collaborator

elasticsearchmachine commented Dec 22, 2024 •
edited
Loading

Build Scans:

Reproduction Line:

./gradlew ":server:test" --tests "org.elasticsearch.index.mapper.AbstractShapeGeometryFieldMapperTests.testCartesianBoundsBlockLoader" -Dtests.seed=FE54B8FD0E47E55E -Dtests.locale=en-IO -Dtests.timezone=America/Puerto_Rico -Druntime.java=23

Applicable branches:
8.x

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: geometries[20] ('polygon=linearring(x=[-1.2879859936655625E35, -1.7732908735768593E35, -1.8468340192899126E35, -1.9172393336265884E35, -2.3696137210352532E35, -2.5285349173073343E35, -3.8256197702616444E35, -4.1676041...') wasn't extracted correctly
Expected: "    BBOX (-5.567426758548179E35, -1.268935185848195E35, 2.2139327585395992E35, -2.273939376546278E35)"
     but: was <BBOX (-3.4028234663852886E38, 1.0987931513188041E38, 7.643878356870293E37, -3.4028234663852886E38)>

Issue Reasons:

  • [8.x] 2 consecutive failures in test testCartesianBoundsBlockLoader
  • [8.x] 10 failures in test testCartesianBoundsBlockLoader (4.5% fail rate in 223 executions)
  • [8.x] 2 failures in step ubuntu-2004_platform-support-unix (33.3% fail rate in 6 executions)
  • [8.x] 2 failures in step ubuntu-2204_platform-support-unix (40.0% fail rate in 5 executions)
  • [8.x] 4 failures in pipeline elasticsearch-periodic-platform-support (57.1% fail rate in 7 executions)
  • [8.x] 2 failures in pipeline elasticsearch-periodic (33.3% fail rate in 6 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Analytics/Geo Indexing, search aggregations of geo points and shapes >test-failure Triaged test failures from CI Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) needs:risk Requires assignment of a risk label (low, medium, blocker) labels Dec 22, 2024
@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-analytical-engine (Team:Analytics)

@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch 8.x

Mute Reasons:

  • [8.x] 2 failures in test testCartesianBoundsBlockLoader (2.2% fail rate in 93 executions)

Build Scans:

elasticsearchmachine added a commit that referenced this issue Dec 23, 2024
@elasticsearchmachine
Mute org.elasticsearch.index.mapper.AbstractShapeGeometryFieldMapperT…
7ffbcaf 
…ests testCartesianBoundsBlockLoader #119201
@ivancea ivancea added low-risk An open issue or test failure that is a low risk to future releases and removed needs:risk Requires assignment of a risk label (low, medium, blocker) labels Dec 24, 2024
@ivancea
Copy link
Contributor

ivancea commented Dec 24, 2024

Looks like it also fails in main (Failed in #119263)

@GalLalouche GalLalouche mentioned this issue Dec 24, 2024
Merged
GalLalouche added a commit to GalLalouche/elasticsearch that referenced this issue Dec 26, 2024
@GalLalouche
ESQL: Fix AbstractShapeGeometryFieldMapperTests (elastic#119265)
e74b6ae 
Fixed a bug in AbstractShapeGeometryFieldMapperTests when the directory reader would contain multiple leaves.

Resolves elastic#119201.
elasticsearchmachine pushed a commit that referenced this issue Dec 26, 2024
@GalLalouche
[8.x] ESQL: Fix AbstractShapeGeometryFieldMapperTests (#119265) (#119284
0e0b4fb 
)

* ESQL: Fix AbstractShapeGeometryFieldMapperTests (#119265)

Fixed a bug in AbstractShapeGeometryFieldMapperTests when the directory reader would contain multiple leaves.

Resolves #119201.

* Unmute test
elasticsearchmachine added a commit that referenced this issue Dec 26, 2024
@elasticsearchmachine
Mute org.elasticsearch.index.mapper.AbstractShapeGeometryFieldMapperT…
2fbeccb 
…ests testCartesianBoundsBlockLoader #119201
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 2 consecutive failures in test testCartesianBoundsBlockLoader
  • [main] 7 failures in test testCartesianBoundsBlockLoader (2.0% fail rate in 342 executions)
  • [main] 3 failures in step part-1 (3.0% fail rate in 101 executions)
  • [main] 2 failures in step openjdk21_checkpart1_java-matrix (25.0% fail rate in 8 executions)
  • [main] 2 failures in pipeline elasticsearch-periodic-platform-support (22.2% fail rate in 9 executions)
  • [main] 3 failures in pipeline elasticsearch-pull-request (3.1% fail rate in 97 executions)
  • [main] 2 failures in pipeline elasticsearch-periodic (25.0% fail rate in 8 executions)

Build Scans:

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

@GalLalouche GalLalouche

Labels
:Analytics/Geo Indexing, search aggregations of geo points and shapes low-risk An open issue or test failure that is a low risk to future releases Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test-failure Triaged test failures from CI
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

ESQL: Fix AbstractShapeGeometryFieldMapperTests GalLalouche/elasticsearch
3 participants
@ivancea @GalLalouche @elasticsearchmachine