Skip to content

[CI] HeapAttackIT testLookupExplosion failing #125779

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 Mar 27, 2025 · 2 comments · Fixed by #126874
Closed

[CI] HeapAttackIT testLookupExplosion failing #125779

elasticsearchmachine opened this issue Mar 27, 2025 · 2 comments · Fixed by #126874
Assignees
@nik9000
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 Mar 27, 2025 •
edited
Loading

Build Scans:

Reproduction Line:

./gradlew ":test:external-modules:test-esql-heap-attack:javaRestTest" --tests "org.elasticsearch.xpack.esql.heap_attack.HeapAttackIT.testLookupExplosion" -Dtests.seed=343D03F300F9B039 -Dtests.locale=vun-Latn-TZ -Dtests.timezone=Indian/Chagos -Druntime.java=24

Applicable branches:
9.0

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

org.elasticsearch.client.ResponseException: method [POST], host [http://[::1]:40005], URI [/_query?error_trace=], status line [HTTP/1.1 429 Too Many Requests]
Warnings: [No limit defined, adding default limit of [1000]]
{"error":{"root_cause":[{"type":"circuit_breaking_exception","reason":"[request] Data too large, data for [serialize lookup join response] would be [335303584/319.7mb], which is larger than the limit of [322122547/307.1mb]; for more information, see https://www.elastic.co/docs/troubleshoot/elasticsearch/circuit-breaker-errors?version=master","bytes_wanted":335303584,"bytes_limit":322122547,"durability":"TRANSIENT","stack_trace":"org.elasticsearch.common.breaker.CircuitBreakingException: [request] Data too large, data for [serialize lookup join response] would be [335303584/319.7mb], which is larger than the limit of [322122547/307.1mb]; for more information, see https://www.elastic.co/docs/troubleshoot/elasticsearch/circuit-breaker-errors?version=master\n\tat org.elasti
[truncated]

Issue Reasons:

  • [9.0] 2 failures in test testLookupExplosion (0.4% fail rate in 466 executions)
  • [9.0] 2 failures in pipeline elasticsearch-periodic-platform-support (10.5% fail rate in 19 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added :Analytics/ES|QL AKA ESQL >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 Mar 27, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@luigidellaquila luigidellaquila added the low-risk An open issue or test failure that is a low risk to future releases label Mar 27, 2025
@elasticsearchmachine elasticsearchmachine removed the needs:risk Requires assignment of a risk label (low, medium, blocker) label Mar 27, 2025
@nik9000 nik9000 self-assigned this Mar 27, 2025
@nik9000
Copy link
Member

nik9000 commented Mar 27, 2025

Fair. We should be ok with this 429.

@alex-spies alex-spies mentioned this issue Mar 31, 2025
Closed
nik9000 added a commit to nik9000/elasticsearch that referenced this issue Apr 15, 2025
@nik9000
ESQL: Create fewer documents in lookup tests
4be4493 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

Closes elastic#125913
Closes elastic#125779
@nik9000 nik9000 mentioned this issue Apr 15, 2025
Merged
nik9000 added a commit to nik9000/elasticsearch that referenced this issue Apr 15, 2025
@nik9000
ESQL: Create fewer documents in lookup tests (elastic#126874)
fe3e8df 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

Closes elastic#125913 Closes elastic#125779
nik9000 added a commit to nik9000/elasticsearch that referenced this issue Apr 15, 2025
@nik9000
ESQL: Create fewer documents in lookup tests (elastic#126874)
c5ad177 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

Closes elastic#125913 Closes elastic#125779
nik9000 added a commit to nik9000/elasticsearch that referenced this issue Apr 15, 2025
@nik9000
ESQL: Create fewer documents in lookup tests (elastic#126874)
807c2fb 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

Closes elastic#125913 Closes elastic#125779
afoucret pushed a commit to afoucret/elasticsearch that referenced this issue Apr 16, 2025
@nik9000 @afoucret
ESQL: Create fewer documents in lookup tests (elastic#126874)
31d7b96 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

Closes elastic#125913 Closes elastic#125779
elasticsearchmachine pushed a commit that referenced this issue Apr 16, 2025
@nik9000
ESQL: Create fewer documents in lookup tests (#126874) (#126877)
d71d1b7 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

Closes #125913 Closes #125779
elasticsearchmachine pushed a commit that referenced this issue Apr 16, 2025
@nik9000
ESQL: Create fewer documents in lookup tests (#126874) (#126878)
2d2a3a2 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

Closes #125913 Closes #125779
elasticsearchmachine pushed a commit that referenced this issue Apr 18, 2025
@nik9000
ESQL: Create fewer documents in lookup tests (#126874) (#126879)
e323320 
This lowers the number of documents used to test lookup because we have
a few failures over the last few months. These are all cases that we
expect to *pass* so fewer documents should make them even more likely to
pass.

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

@nik9000 nik9000

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.

ESQL: Create fewer documents in lookup tests nik9000/elasticsearch
3 participants
@nik9000 @luigidellaquila @elasticsearchmachine