[CI] RestEsqlIT.testSuggestedCast (non-snapshot)

[luigidellaquila]

CI Link

https://gradle-enterprise.elastic.co/s/7rcjxa6dus62u/tests/task/:x-pack:plugin:esql:qa:server:single-node:javaRestTest/details/org.elasticsearch.xpack.esql.qa.single_node.RestEsqlIT/testSuggestedCast%20%7BASYNC%7D?top-execution=1

Repro line

./gradlew ":x-pack:plugin:esql:qa:server:single-node:javaRestTest" --tests "org.elasticsearch.xpack.esql.qa.single_node.RestEsqlIT.testSuggestedCast {ASYNC}" -Dtests.seed=F76DEA6025845555 -Dbuild.snapshot=false -Dtests.jvm.argline="-Dbuild.snapshot=false" -Dlicense.key=x-pack/license-tools/src/test/resources/public.key -Dtests.locale=ses-ML -Dtests.timezone=Asia/Bishkek -Druntime.java=24

Does it reproduce?

Didn't try

Applicable branches

main

Failure history

No response

Failure excerpt

java.lang.AssertionError: |  
-- | --
Expected: <[{suggested_cast=date_nanos, original_types=[date_nanos, datetime], name=my_field, type=unsupported}]> |  
but: was <[{name=my_field, type=date_nanos}]>

I can't reproduce it locally, due to known problems with ML module and release tests

@limotova this only seems to fail in release tests, and I'm not sure if it's still supposed to be snapshot-only. Could you please have a look?

[elasticsearchmachine]

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

[alex-spies]

@limotova , I hope it's okay I just go ahead and assign you for the time being, since you have the best context on this test.

Please feel free to bounce this back if you think some other change caused this or if someone else can better help with this!

[alex-spies]

I took another look - actually, it could be that the implicit casting of date/nanos union types isn't correctly turned off in release builds. I was sure we turned that off in #127797, but going back I can only see the snapshot-only switch on the capability, not on the analyzer rule.

@fang-xing-esql , could you have a look at the behavior of the implicit date/nanos casting in release scenarios?

[alex-spies]

Argh, yes, that's exactly what happened. I can confirm that implicit casting of dates/nanos isn't correctly guarded behind SNAPSHOT.

I have no idea how the release tests didn't raise an issue for so long.