Skip to content

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

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

Closed
elasticsearchmachine opened this issue Mar 7, 2025 · 5 comments · Fixed by #124685
Closed
Assignees
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 7, 2025

Build Scans:
(9.0)

(main)

Reproduction Line:

./gradlew ":x-pack:plugin:esql:test" --tests "org.elasticsearch.xpack.esql.expression.function.aggregate.ValuesTests.testGroupingAggregate {TestCase=<<no alt geo_shape>s>}" -Dtests.seed=94CED618DAEC14A8 -Dtests.locale=jgo-CM -Dtests.timezone=Pacific/Truk -Druntime.java=24

Applicable branches:
main, 9.0

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

java.lang.OutOfMemoryError: Java heap space

Issue Reasons:

  • [9.0] 2 failures in test testGroupingAggregate {TestCase=<s>} (0.7% fail rate in 267 executions)
  • [9.0] 2 failures in pipeline elasticsearch-periodic-platform-support (25.0% fail rate in 8 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 7, 2025
@elasticsearchmachine
Copy link
Collaborator Author

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

@astefan
Copy link
Contributor

astefan commented Mar 7, 2025

@nik9000 could you have a look at this one, you've touched this one recently.

For the reference, this is the relevant part of the logs:

REPRODUCE WITH: ./gradlew ":x-pack:plugin:esql:test" --tests "org.elasticsearch.xpack.esql.expression.function.aggregate.ValuesTests.testGroupingAggregate {TestCase=<<no alt geo_shape>s>}" -Dtests.seed=E2F62EF29DBDBC33 -Dtests.locale=jgo-Latn-CM -Dtests.timezone=Australia/Queensland -Druntime.java=23

ValuesTests > testGroupingAggregate {TestCase=<<no alt geo_shape>s>} FAILED
    java.lang.OutOfMemoryError: Java heap space
        at __randomizedtesting.SeedInfo.seed([E2F62EF29DBDBC33:6174681FFFB5B3D0]:0)
        at org.elasticsearch.common.util.BigByteArray.get(BigByteArray.java:73)
        at org.elasticsearch.common.util.MockBigArrays$ByteArrayWrapper.get(MockBigArrays.java:399)
        at org.elasticsearch.common.util.BytesRefArray.get(BytesRefArray.java:106)
        at org.elasticsearch.compute.data.BytesRefArrayVector.getBytesRef(BytesRefArrayVector.java:72)
        at org.elasticsearch.compute.data.BytesRefArrayBlock.getBytesRef(BytesRefArrayBlock.java:98)
        at org.elasticsearch.compute.data.BlockUtils.valueAtOffset(BlockUtils.java:278)
        at org.elasticsearch.compute.data.BlockUtils.toJavaObject(BlockUtils.java:270)
        at org.elasticsearch.xpack.esql.expression.function.AbstractAggregationTestCase.lambda$extractResultsFromAggregator$6(AbstractAggregationTestCase.java:365)
        at org.elasticsearch.xpack.esql.expression.function.AbstractAggregationTestCase$$Lambda/0x000000002481bc50.apply(Unknown Source)
        at java.base/java.util.stream.IntPipeline$1$1.accept(IntPipeline.java:180)
        at java.base/java.util.stream.Streams$RangeIntSpliterator.forEachRemaining(Streams.java:104)
        at java.base/java.util.Spliterator$OfInt.forEachRemaining(Spliterator.java:712)
        at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:570)
        at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:560)
        at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:636)
        at java.base/java.util.stream.AbstractPipeline.evaluateToArrayNode(AbstractPipeline.java:291)
        at java.base/java.util.stream.ReferencePipeline.toArray(ReferencePipeline.java:656)
        at java.base/java.util.stream.ReferencePipeline.toArray(ReferencePipeline.java:662)
        at java.base/java.util.stream.ReferencePipeline.toList(ReferencePipeline.java:667)
        at org.elasticsearch.xpack.esql.expression.function.AbstractAggregationTestCase.extractResultsFromAggregator(AbstractAggregationTestCase.java:366)
        at org.elasticsearch.xpack.esql.expression.function.AbstractAggregationTestCase.aggregateGroupingSingleMode(AbstractAggregationTestCase.java:195)
        at org.elasticsearch.xpack.esql.expression.function.AbstractAggregationTestCase$$Lambda/0x0000000024819a98.accept(Unknown Source)
        at org.elasticsearch.xpack.esql.expression.function.AbstractAggregationTestCase.resolveExpression(AbstractAggregationTestCase.java:328)
        at org.elasticsearch.xpack.esql.expression.function.AbstractAggregationTestCase.testGroupingAggregate(AbstractAggregationTestCase.java:146)
        at java.base/java.lang.invoke.LambdaForm$DMH/0x0000000024000c00.invokeVirtual(LambdaForm$DMH)
        at java.base/java.lang.invoke.LambdaForm$MH/0x0000000024846800.invoke(LambdaForm$MH)
        at java.base/java.lang.invoke.Invokers$Holder.invokeExact_MT(Invokers$Holder)
        at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invokeImpl(DirectMethodHandleAccessor.java:153)
        at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:103)
        at java.base/java.lang.reflect.Method.invoke(Method.java:580)
        at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1763)
        at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:946)

@idegtiarenko idegtiarenko added the low-risk An open issue or test failure that is a low risk to future releases label Mar 10, 2025
@elasticsearchmachine elasticsearchmachine removed the needs:risk Requires assignment of a risk label (low, medium, blocker) label Mar 10, 2025
@idegtiarenko
Copy link
Contributor

I am assigning low risk as this is "java.lang.OutOfMemoryError: Java heap space" during test execution and is likely require test tuning rather than actual production code change. Please feel free to change the priority if you see otherwise.

@idegtiarenko
Copy link
Contributor

Please note, this issue was observed in main too

@nik9000
Copy link
Member

nik9000 commented Mar 12, 2025

@craigtaverner added this support recently. I believe this is just a "too large" bit of test data. checking.

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

Closes elastic#124330
nik9000 added a commit to nik9000/elasticsearch that referenced this issue Mar 12, 2025
The test was generating too much data

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

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

Closes elastic#124330
jfreden pushed a commit to jfreden/elasticsearch that referenced this issue 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 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
Development

Successfully merging a pull request may close this issue.

4 participants