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] MemoryTrackingTDigestArraysTests testIntBulkSet failing #113744

Closed
elasticsearchmachine opened this issue Sep 29, 2024 · 2 comments · Fixed by #113934
Closed

[CI] MemoryTrackingTDigestArraysTests testIntBulkSet failing #113744

elasticsearchmachine opened this issue Sep 29, 2024 · 2 comments · Fixed by #113934
Assignees
@ivancea
Labels
:Analytics/ES|QL AKA ESQL 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 Sep 29, 2024 •
edited
Loading

Build Scans:

Reproduction Line:

./gradlew ":server:test" --tests "org.elasticsearch.search.aggregations.metrics.MemoryTrackingTDigestArraysTests.testIntBulkSet" -Dtests.seed=D7098FAD3010FB55 -Dtests.locale=mgh-MZ -Dtests.timezone=America/Monterrey -Druntime.java=22

Applicable branches:
main

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.AssertionError: max must be >= min: 0, -1

Issue Reasons:

  • [main] 3 failures in test testIntBulkSet (0.5% fail rate in 550 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :StorageEngine/Mapping The storage related side of mappings >test-failure Triaged test failures from CI Team:StorageEngine needs:risk Requires assignment of a risk label (low, medium, blocker) labels Sep 29, 2024
@elasticsearchmachine
Copy link
Collaborator Author

Pinging @elastic/es-storage-engine (Team:StorageEngine)

@dnhatn dnhatn added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL and removed Team:StorageEngine :StorageEngine/Mapping The storage related side of mappings labels Sep 29, 2024
@elasticsearchmachine
Copy link
Collaborator Author

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

@ivancea ivancea mentioned this issue Oct 2, 2024
Merged
@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 Oct 2, 2024
ivancea added a commit to ivancea/elasticsearch that referenced this issue Oct 2, 2024
@ivancea
Fix MemoryTracking...ArraysTests bulkSet errors (elastic#113934)
9bbf529 
Fixes elastic#113914 Fixes
elastic#113744

Some random numbers in the tests were leading to negative indexes in
some cases. Just refactored them a bit so no impossible values are
generated. And it's now a bit more _readable_...
gmarouli pushed a commit to gmarouli/elasticsearch that referenced this issue Oct 3, 2024
@ivancea @gmarouli
Fix MemoryTracking...ArraysTests bulkSet errors (elastic#113934)
58dc49c 
Fixes elastic#113914 Fixes
elastic#113744

Some random numbers in the tests were leading to negative indexes in
some cases. Just refactored them a bit so no impossible values are
generated. And it's now a bit more _readable_...
matthewabbott pushed a commit to matthewabbott/elasticsearch that referenced this issue Oct 4, 2024
@ivancea @matthewabbott
Fix MemoryTracking...ArraysTests bulkSet errors (elastic#113934)
6c97e8f 
Fixes elastic#113914 Fixes
elastic#113744

Some random numbers in the tests were leading to negative indexes in
some cases. Just refactored them a bit so no impossible values are
generated. And it's now a bit more _readable_...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@ivancea ivancea

Labels
:Analytics/ES|QL AKA ESQL 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.

Fix MemoryTracking...ArraysTests bulkSet errors ivancea/elasticsearch
3 participants
@ivancea @dnhatn @elasticsearchmachine