Skip to content

Conversation

@alex-spies
Copy link
Contributor

When a new type is added, it is marked as under construction and considered always supported when running tests on SNAPSHOT builds.

This is problematic for existing ENRICH tests because they correctly use DATE_RANGE fields, which we are trying to add support to. When turning DATE_RANGE from unsupported to under construction, our bwc tests break because old nodes, of course, can't even deserialize the new DATE_RANGE data type.

Add the notion of a createdVersion for new data types. This is the first version from which the type can be used in SNAPSHOT builds and tests; for released types, this version is additional (and generally different) to the supported version, which is the first version on which production builds will consider a type as supported.

@alex-spies alex-spies added >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build :Analytics/ES|QL AKA ESQL v9.2.1 v9.3.0 labels Oct 31, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Oct 31, 2025
Copy link
Member

@nik9000 nik9000 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Makes sense to me and should make our situation better.

@nik9000 nik9000 self-assigned this Nov 11, 2025
@alex-spies alex-spies marked this pull request as ready for review December 2, 2025 09:27
@alex-spies
Copy link
Contributor Author

Will update and merge after #137431.

@alex-spies alex-spies added the auto-backport Automatically create backport pull requests when merged label Dec 2, 2025
@alex-spies alex-spies added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Dec 2, 2025
@alex-spies
Copy link
Contributor Author

Unrelated release test failure: #138946
Unrelated serverless checks failures: https://github.com/elastic/elasticsearch-serverless/issues/4940

Let's get these muted; at least the serverless checks better be passing. (The only release tests that matter are the AllFieldsSupportedITs, and they're fine.)

@craigtaverner craigtaverner enabled auto-merge (squash) December 4, 2025 23:01
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 test-release Trigger CI checks against release build v9.2.3 v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants