Skip to content

[CI] FilteredGroupingAggregatorFunctionTests testSimpleCircuitBreaking failing #127833

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 May 7, 2025 · 3 comments · Fixed by #127909
Closed

[CI] FilteredGroupingAggregatorFunctionTests testSimpleCircuitBreaking failing #127833

elasticsearchmachine opened this issue May 7, 2025 · 3 comments · Fixed by #127909
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 May 7, 2025 •
edited
Loading

Build Scans:

Reproduction Line:

./gradlew ":x-pack:plugin:esql:compute:test" --tests "org.elasticsearch.compute.aggregation.FilteredGroupingAggregatorFunctionTests.testSimpleCircuitBreaking" -Dtests.seed=40865A7C7F748006 -Dtests.locale=de-DE -Dtests.timezone=Pacific/Ponape -Druntime.java=24

Applicable branches:
main

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.IllegalArgumentException: expected runnable to break under a limit of 130.2kb bytes

Issue Reasons:

  • [main] 7 failures in test testSimpleCircuitBreaking (2.9% fail rate in 242 executions)
  • [main] 3 failures in step part-3 (2.9% fail rate in 102 executions)
  • [main] 4 failures in pipeline elasticsearch-pull-request (4.0% fail rate in 100 executions)

Note:
This issue was created using new test triage automation. Please report issues or feedback to es-delivery.
@elasticsearchmachine elasticsearchmachine added >test-failure Triaged test failures from CI :Analytics/ES|QL AKA ESQL labels May 7, 2025
elasticsearchmachine added a commit that referenced this issue May 7, 2025
@elasticsearchmachine
Mute org.elasticsearch.compute.aggregation.FilteredGroupingAggregator…
044f856 
…FunctionTests testSimpleCircuitBreaking #127833
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 5 failures in test testSimpleCircuitBreaking (2.1% fail rate in 235 executions)
  • [main] 3 failures in step part-3 (2.9% fail rate in 102 executions)
  • [main] 4 failures in pipeline elasticsearch-pull-request (4.0% fail rate in 101 executions)

Build Scans:

@elasticsearchmachine elasticsearchmachine added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) needs:risk Requires assignment of a risk label (low, medium, blocker) labels May 7, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@alex-spies
Copy link
Contributor

@ivancea , I see you made some changes to FilteredGroupingAggregatorFunction today; do you think you could take a look at this test failure?

@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 May 7, 2025
@ivancea ivancea mentioned this issue May 8, 2025
Merged
ywangd pushed a commit to ywangd/elasticsearch that referenced this issue May 9, 2025
@elasticsearchmachine @ywangd
Mute org.elasticsearch.compute.aggregation.FilteredGroupingAggregator…
d1ddc25 
…FunctionTests testSimpleCircuitBreaking elastic#127833
afoucret pushed a commit to afoucret/elasticsearch that referenced this issue May 9, 2025
@elasticsearchmachine @afoucret
Mute org.elasticsearch.compute.aggregation.FilteredGroupingAggregator…
8d600e9 
…FunctionTests testSimpleCircuitBreaking elastic#127833
ivancea added a commit that referenced this issue May 9, 2025
@ivancea
ESQL: Fix block randomization on circuit breaking tests (#127909)
bccd6f2 
Fixes #127833
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.

ESQL: Fix block randomization on circuit breaking tests ivancea/elasticsearch
3 participants
@ivancea @elasticsearchmachine @alex-spies