Skip to content

[CI] LogicalPlanOptimizerTests class failing #118721

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 Dec 14, 2024 · 3 comments · Fixed by #118742
Closed

[CI] LogicalPlanOptimizerTests class failing #118721

elasticsearchmachine opened this issue Dec 14, 2024 · 3 comments · Fixed by #118742
Assignees
@alex-spies
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 Dec 14, 2024 •
edited
Loading

Build Scans:

Reproduction Line:

./gradlew ":x-pack:plugin:esql:test" --tests "org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests.testLookupJoinPushDownFilterOnJoinKeyWithRename" -Dtests.seed=50923C2E21EDE0B2 -Dbuild.snapshot=false -Dtests.jvm.argline="-Dbuild.snapshot=false" -Dlicense.key=x-pack/license-tools/src/test/resources/public.key -Dtests.locale=fr-Latn-FR -Dtests.timezone=Europe/Kiev -Druntime.java=23

Applicable branches:
8.x

Reproduces locally?:
N/A

Failure History:
See dashboard

Failure Message:

org.elasticsearch.xpack.esql.parser.ParsingException: line 3:3: mismatched input 'LOOKUP' expecting {'dissect', 'drop', 'enrich', 'eval', 'grok', 'keep', 'limit', 'mv_expand', 'rename', 'sort', 'stats', 'where'}

Issue Reasons:

  • [8.x] 6 consecutive failures in step release-tests
  • [8.x] 39 failures in class org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests (15.7% fail rate in 248 executions)
  • [8.x] 7 failures in step release-tests (87.5% fail rate in 8 executions)
  • [8.x] 6 failures in step part3 (24.0% fail rate in 25 executions)
  • [8.x] 2 failures in step part-3-arm (50.0% fail rate in 4 executions)
  • [8.x] 24 failures in step part-3 (61.5% fail rate in 39 executions)
  • [8.x] 5 failures in pipeline elasticsearch-periodic (83.3% fail rate in 6 executions)
  • [8.x] 6 failures in pipeline elasticsearch-intake (24.0% fail rate in 25 executions)
  • [8.x] 21 failures in pipeline elasticsearch-pull-request (60.0% fail rate in 35 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 labels Dec 14, 2024
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch main

Mute Reasons:

  • [main] 2 consecutive failures in step release-tests
  • [main] 10 failures in class org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests (1.1% fail rate in 873 executions)
  • [main] 2 failures in step release-tests (14.3% fail rate in 14 executions)
  • [main] 6 failures in step part-3 (1.3% fail rate in 457 executions)
  • [main] 2 failures in step part3 (1.3% fail rate in 153 executions)
  • [main] 2 failures in pipeline elasticsearch-periodic (16.7% fail rate in 12 executions)
  • [main] 6 failures in pipeline elasticsearch-pull-request (1.3% fail rate in 457 executions)
  • [main] 2 failures in pipeline elasticsearch-intake (1.3% fail rate in 153 executions)

Build Scans:

elasticsearchmachine added a commit that referenced this issue Dec 14, 2024
@elasticsearchmachine
Mute org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests…
fee6165 
… org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests #118721
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Dec 14, 2024
@elasticsearchmachine
Copy link
Collaborator Author

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

@elasticsearchmachine elasticsearchmachine added the needs:risk Requires assignment of a risk label (low, medium, blocker) label Dec 14, 2024
@elasticsearchmachine
Copy link
Collaborator Author

This has been muted on branch 8.x

Mute Reasons:

  • [8.x] 6 consecutive failures in step release-tests
  • [8.x] 39 failures in class org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests (15.7% fail rate in 248 executions)
  • [8.x] 7 failures in step release-tests (87.5% fail rate in 8 executions)
  • [8.x] 6 failures in step part3 (24.0% fail rate in 25 executions)
  • [8.x] 2 failures in step part-3-arm (50.0% fail rate in 4 executions)
  • [8.x] 24 failures in step part-3 (61.5% fail rate in 39 executions)
  • [8.x] 5 failures in pipeline elasticsearch-periodic (83.3% fail rate in 6 executions)
  • [8.x] 6 failures in pipeline elasticsearch-intake (24.0% fail rate in 25 executions)
  • [8.x] 21 failures in pipeline elasticsearch-pull-request (60.0% fail rate in 35 executions)

Build Scans:

elasticsearchmachine added a commit that referenced this issue Dec 15, 2024
@elasticsearchmachine
Mute org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests…
82cc8d8 
… org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests #118721
@alex-spies alex-spies self-assigned this Dec 16, 2024
@alex-spies alex-spies 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 Dec 16, 2024
@alex-spies alex-spies mentioned this issue Dec 16, 2024
Merged
alex-spies added a commit to alex-spies/elasticsearch that referenced this issue Dec 16, 2024
@alex-spies
ESQL: Fix lookup optimizer tests on release (elastic#118742)
5250e07 
Fix elastic#118721

* Skip corresponding optimizer tests if `LOOKUP JOIN` is disabled.
* Enable LogicalPlanOptimizerTests again.

(cherry picked from commit bb8503a)

# Conflicts:
#	muted-tests.yml
elasticsearchmachine pushed a commit that referenced this issue Dec 16, 2024
@alex-spies
ESQL: Fix lookup optimizer tests on release (#118742) (#118750)
3080975 
Fix #118721

* Skip corresponding optimizer tests if `LOOKUP JOIN` is disabled.
* Enable LogicalPlanOptimizerTests again.

(cherry picked from commit bb8503a)

# Conflicts:
#	muted-tests.yml
maxhniebergall pushed a commit to maxhniebergall/elasticsearch that referenced this issue Dec 16, 2024
@elasticsearchmachine @maxhniebergall
Mute org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests…
a341a79 
… org.elasticsearch.xpack.esql.optimizer.LogicalPlanOptimizerTests elastic#118721
maxhniebergall pushed a commit to maxhniebergall/elasticsearch that referenced this issue Dec 16, 2024
@alex-spies @maxhniebergall
ESQL: Fix lookup optimizer tests on release (elastic#118742) (elastic…
2fa7aac 
…#118750)

Fix elastic#118721

* Skip corresponding optimizer tests if `LOOKUP JOIN` is disabled.
* Enable LogicalPlanOptimizerTests again.

(cherry picked from commit bb8503a)

# Conflicts:
#	muted-tests.yml
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@alex-spies alex-spies

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 lookup optimizer tests on release alex-spies/elasticsearch
2 participants
@elasticsearchmachine @alex-spies