Skip to content
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

ESQL: Readd union types capability to indirect tests #119776

Merged

Conversation

GalLalouche
Copy link
Contributor

This PR re-adds the union types capabilities to the CSV-spec tests using indirect union types, which were removed in #119273.

@GalLalouche GalLalouche added >test Issues or PRs that are addressing/adding tests Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL v8.18.0 labels Jan 8, 2025
@GalLalouche GalLalouche requested a review from alex-spies January 8, 2025 16:38
@GalLalouche GalLalouche force-pushed the tests/readd_union_types_cap_8.x branch from 68bad10 to ffde611 Compare January 8, 2025 16:38
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@alex-spies alex-spies left a comment

Choose a reason for hiding this comment

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

LGTM, thanks @GalLalouche !

@GalLalouche GalLalouche enabled auto-merge (squash) January 8, 2025 16:44
@alex-spies alex-spies added the test-full-bwc Trigger full BWC version matrix tests label Jan 8, 2025
@GalLalouche GalLalouche merged commit 43e4404 into elastic:8.x Jan 8, 2025
15 checks passed
elasticsearchmachine pushed a commit that referenced this pull request Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
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 test-full-bwc Trigger full BWC version matrix tests v8.18.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants