Skip to content

ESQL: Fix a test oom #124685

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.

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

Merged
merged 1 commit into from
Mar 12, 2025
Merged

ESQL: Fix a test oom #124685

merged 1 commit into from
Mar 12, 2025

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented Mar 12, 2025

The test was generating too much data

Closes #124330

The test was generating too much data

Closes elastic#124330
@nik9000 nik9000 added >test Issues or PRs that are addressing/adding tests auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) auto-backport Automatically create backport pull requests when merged :Analytics/ES|QL AKA ESQL v9.0.0 v9.1.0 labels Mar 12, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 12, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@nik9000
Copy link
Member Author

nik9000 commented Mar 12, 2025

I reproduced the issue in #124330 locally by running a few thousand times. Then I dropped the data size and have run for a few hours and didn't see the issue.

@elasticsearchmachine elasticsearchmachine merged commit 7e746ce into elastic:main Mar 12, 2025
17 checks passed
@nik9000 nik9000 deleted the fix_124330 branch March 12, 2025 23:36
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
9.0

nik9000 added a commit to nik9000/elasticsearch that referenced this pull request Mar 12, 2025
The test was generating too much data

Closes elastic#124330
elasticsearchmachine pushed a commit that referenced this pull request Mar 13, 2025
The test was generating too much data

Closes #124330
albertzaharovits pushed a commit to albertzaharovits/elasticsearch that referenced this pull request Mar 13, 2025
The test was generating too much data

Closes elastic#124330
jfreden pushed a commit to jfreden/elasticsearch that referenced this pull request Mar 13, 2025
The test was generating too much data

Closes elastic#124330
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL auto-backport Automatically create backport pull requests when merged auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.0.0 v9.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] ValuesTests testGroupingAggregate {TestCase=<<no alt geo_shape>s>} failing
2 participants