Skip to content

Update expected esql.features#144332

Merged
burqen merged 2 commits intoelastic:mainfrom
burqen:ap/2026.03.16.fix-XPackRestIT
Mar 17, 2026
Merged

Update expected esql.features#144332
burqen merged 2 commits intoelastic:mainfrom
burqen:ap/2026.03.16.fix-XPackRestIT

Conversation

@burqen
Copy link
Copy Markdown
Contributor

@burqen burqen commented Mar 16, 2026

This has already been updated for "Basic ESQL usage output (telemetry) snapshot version". Updating non-snapshot version to expect the same value.

Fixes this test failure:

REPRODUCE WITH: ./gradlew ":x-pack:plugin:yamlRestTest" --tests "org.elasticsearch.xpack.test.rest.XPackRestIT.test {p0=esql/60_usage/Basic ESQL usage output (telemetry) non-snapshot version}" -Dtests.seed=F5DB966DB5E569ED -Dbuild.snapshot=false -Dtests.jvm.argline="-Dbuild.snapshot=false" -Dlicense.key=x-pack/license-tools/src/test/resources/public.key -Dtests.locale=ff-Adlm-GN -Dtests.timezone=Pacific/Pohnpei -Druntime.java=25
--
 
XPackRestIT > test > test {p0=esql/60_usage/Basic ESQL usage output (telemetry) non-snapshot version} FAILED
java.lang.AssertionError: org.junit.internal.runners.model.MultipleFailureException: There were 2 errors:
java.lang.AssertionError(field [esql.features] doesn't have length [35]
Expected: <35>
but: was <37>)
java.lang.AssertionError(field [esql.functions.json_extract] exists, but should not.
Expected: null
but: was <0>)
at __randomizedtesting.SeedInfo.seed([F5DB966DB5E569ED:7D8FA9B71B190415]:0)
at org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase$ESClientYamlSuiteErrorCollector.verify(ESClientYamlSuiteTestCase.java:590)
at org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase.test(ESClientYamlSuiteTestCase.java:504)
at java.base/jdk.internal.reflect.DirectMethodHandleAccessor.invoke(DirectMethodHandleAccessor.java:104)
at java.base/java.lang.reflect.Method.invoke(Method.java:565)
at com.carrotsearch.randomizedtesting.RandomizedRunner.invoke(RandomizedRunner.java:1763)
at com.carrotsearch.randomizedtesting.RandomizedRunner$8.evaluate(RandomizedRunner.java:946)
at com.carrotsearch.randomizedtesting.RandomizedRunner$9.evaluate(RandomizedRunner.java:982)
at com.carrotsearch.randomizedtesting.RandomizedRunner$10.evaluate(RandomizedRunner.java:996)
at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at org.junit.rules.TestWatcher$1.evaluate(TestWatcher.java:61)
at org.apache.lucene.tests.util.TestRuleSetupTeardownChained$1.evaluate(TestRuleSetupTeardownChained.java:48)
at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
at org.apache.lucene.tests.util.TestRuleThreadAndTestName$1.evaluate(TestRuleThreadAndTestName.java:45)
at org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60)
at org.apache.lucene.tests.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:44)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:390)
at com.carrotsearch.randomizedtesting.ThreadLeakControl.forkTimeoutingTask(ThreadLeakControl.java:843)
at com.carrotsearch.randomizedtesting.ThreadLeakControl$3.evaluate(ThreadLeakControl.java:490)
at com.carrotsearch.randomizedtesting.RandomizedRunner.runSingleTest(RandomizedRunner.java:955)
at com.carrotsearch.randomizedtesting.RandomizedRunner$5.evaluate(RandomizedRunner.java:840)
at com.carrotsearch.randomizedtesting.RandomizedRunner$6.evaluate(RandomizedRunner.java:891)
at com.carrotsearch.randomizedtesting.RandomizedRunner$7.evaluate(RandomizedRunner.java:902)
at org.elasticsearch.test.cluster.local.DefaultLocalElasticsearchCluster$1.evaluate(DefaultLocalElasticsearchCluster.java:50)
at org.elasticsearch.xpack.inference.MlModelServer$1.evaluate(MlModelServer.java:154)
at org.elasticsearch.cluster.metadata.TemplateDecoratorRule$1.evaluate(TemplateDecoratorRule.java:48)
at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
at org.apache.lucene.tests.util.TestRuleStoreClassName$1.evaluate(TestRuleStoreClassName.java:38)
at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
at com.carrotsearch.randomizedtesting.rules.NoShadowingOrOverridesOnMethodsRule$1.evaluate(NoShadowingOrOverridesOnMethodsRule.java:40)
at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
at org.apache.lucene.tests.util.TestRuleAssertionsRequired$1.evaluate(TestRuleAssertionsRequired.java:53)
at org.apache.lucene.tests.util.AbstractBeforeAfterRule$1.evaluate(AbstractBeforeAfterRule.java:43)
at org.apache.lucene.tests.util.TestRuleMarkFailure$1.evaluate(TestRuleMarkFailure.java:44)
at org.apache.lucene.tests.util.TestRuleIgnoreAfterMaxFailures$1.evaluate(TestRuleIgnoreAfterMaxFailures.java:60)
at org.apache.lucene.tests.util.TestRuleIgnoreTestSuites$1.evaluate(TestRuleIgnoreTestSuites.java:47)
at org.junit.rules.RunRules.evaluate(RunRules.java:20)
at com.carrotsearch.randomizedtesting.rules.StatementAdapter.evaluate(StatementAdapter.java:36)
at com.carrotsearch.randomizedtesting.ThreadLeakControl$StatementRunner.run(ThreadLeakControl.java:390)
at com.carrotsearch.randomizedtesting.ThreadLeakControl.lambda$forkTimeoutingTask$0(ThreadLeakControl.java:850)
at java.base/java.lang.Thread.run(Thread.java:1474)
 
Caused by:
org.junit.internal.runners.model.MultipleFailureException: There were 2 errors:
java.lang.AssertionError(field [esql.features] doesn't have length [35]
Expected: <35>
but: was <37>)
java.lang.AssertionError(field [esql.functions.json_extract] exists, but should not.
Expected: null
but: was <0>)
at app//org.junit.runners.model.MultipleFailureException.assertEmpty(MultipleFailureException.java:102)
at app//org.junit.rules.ErrorCollector.verify(ErrorCollector.java:42)
at app//org.elasticsearch.test.rest.yaml.ESClientYamlSuiteTestCase$ESClientYamlSuiteErrorCollector.verify(ESClientYamlSuiteTestCase.java:588)
... 44 more

@burqen
Update expected esql.features
ec53df1 
This has already been updated for "Basic ESQL usage output (telemetry)
snapshot version". Updating non-snapshot version to expect the same
value.
@burqen burqen added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL v9.4.0 labels Mar 16, 2026
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Mar 16, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

elasticsearchmachine commented Mar 16, 2026

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

@burqen burqen requested a review from ncordon March 16, 2026 15:32
@burqen
Copy link
Copy Markdown
Contributor Author

burqen commented Mar 16, 2026

Hi @ncordon ! Do you mind having a look at this change. From the git blame you seem to have some relevant experience in this area :)

@burqen burqen mentioned this pull request Mar 16, 2026
Merged
@burqen
Copy link
Copy Markdown
Contributor Author

burqen commented Mar 17, 2026

Tests are green but github UI doesn't recognize it. Merging.

@burqen burqen merged commit 375616b into elastic:main Mar 17, 2026
36 checks passed
michalborek pushed a commit to michalborek/elasticsearch that referenced this pull request Mar 23, 2026
@burqen @michalborek
Update expected esql.features (elastic#144332)
e131fa6 
This has already been updated for "Basic ESQL usage output (telemetry)
snapshot version". Updating non-snapshot version to expect the same
value.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@idegtiarenko idegtiarenko idegtiarenko approved these changes

@ncordon ncordon ncordon approved these changes

Assignees

No one assigned

Labels

:Analytics/ES|QL AKA ESQL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) >test Issues or PRs that are addressing/adding tests v9.4.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@burqen @elasticsearchmachine @idegtiarenko @ncordon