Skip to content

[8.19] Fix MixedClusterEsqlSpecIT 8.11 BWC #129841

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

Merged
merged 5 commits into from
Jun 26, 2025
Merged

Conversation

afoucret
Copy link
Contributor

@afoucret afoucret commented Jun 23, 2025

Fix for #129674
Fix for #129596

@afoucret afoucret added >test Issues or PRs that are addressing/adding tests :Analytics/ES|QL AKA ESQL v8.19.0 labels Jun 23, 2025
@elasticsearchmachine elasticsearchmachine added the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Jun 23, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@afoucret afoucret requested a review from ivancea June 23, 2025 09:58
@afoucret afoucret changed the title Fix MixedClusterEsqlSpecIT 8.11 BWC [8.19] Fix MixedClusterEsqlSpecIT 8.11 BWC Jun 23, 2025
Copy link
Contributor

@ivancea ivancea left a comment

Choose a reason for hiding this comment

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

:shipit:

@@ -151,6 +151,13 @@ protected boolean supportsAsync() {
return clusterHasFeature(ASYNC_QUERY_FEATURE_ID); // the Async API was introduced in 8.13.0
}

@After
public void wipeInferenceEndpoints() throws IOException {
if (supportsAsync()) {
Copy link
Contributor

Choose a reason for hiding this comment

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

I suppose those inference endpoints were added between 8.11 and 8.13?

@@ -151,6 +151,13 @@ protected boolean supportsAsync() {
return clusterHasFeature(ASYNC_QUERY_FEATURE_ID); // the Async API was introduced in 8.13.0
}

@After
Copy link
Contributor

Choose a reason for hiding this comment

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

Some reason to have it in the @After now? As it was working before in the @AfterClass

@afoucret afoucret enabled auto-merge (squash) June 26, 2025 12:16
@afoucret afoucret merged commit 79f8140 into elastic:8.19 Jun 26, 2025
22 checks passed
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 v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants