Skip to content

Conversation

@craigtaverner
Copy link
Contributor

@craigtaverner craigtaverner commented Sep 9, 2025

Three PRs have just been merged that cause failures in test-release, and a third PR changes ES|QL railroad diagrams for another recently released change to KNN (also a snapshot-only feature).

This PR attempts to fix all in one, because if we do this in separate PRs they will fail due to the failures cause by the other PRs.

@craigtaverner craigtaverner added >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch :Search Relevance/ES|QL Search functionality in ES|QL labels Sep 9, 2025
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-relevance (Team:Search Relevance)

Copy link
Member

@carlosdelest carlosdelest left a comment

Choose a reason for hiding this comment

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

Thanks for fixing @craigtaverner !

@craigtaverner craigtaverner changed the title Require capability for DECAY function to prevent running in release tests Fix failing release tests (Require capability for DECAY function and SET command) Sep 9, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Sep 9, 2025

ℹ️ Important: Docs version tagging

👋 Thanks for updating the docs! Just a friendly reminder that our docs are now cumulative. This means all 9.x versions are documented on the same page and published off of the main branch, instead of creating separate pages for each minor version.

We use applies_to tags to mark version-specific features and changes.

Expand for a quick overview

When to use applies_to tags:

✅ At the page level to indicate which products/deployments the content applies to (mandatory)
✅ When features change state (e.g. preview, ga) in a specific version
✅ When availability differs across deployments and environments

What NOT to do:

❌ Don't remove or replace information that applies to an older version
❌ Don't add new information that applies to a specific version without an applies_to tag
❌ Don't forget that applies_to tags can be used at the page, section, and inline level

🤔 Need help?

@craigtaverner craigtaverner added Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) :Analytics/ES|QL AKA ESQL labels Sep 9, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@luigidellaquila luigidellaquila left a comment

Choose a reason for hiding this comment

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

If the CI is happy, I'm happy as well.

Thanks @craigtaverner!

@craigtaverner craigtaverner added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Sep 9, 2025
Copy link
Contributor

@timgrein timgrein left a comment

Choose a reason for hiding this comment

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

Thank you for the fix!

@craigtaverner craigtaverner changed the title Fix failing release tests (Require capability for DECAY function and SET command) Fix failing release tests (Require capability for DECAY function and SET and INLINESTATS commands) Sep 9, 2025
@bpintea
Copy link
Contributor

bpintea commented Sep 9, 2025

Thanks, Craig!

@elasticsearchmachine elasticsearchmachine merged commit 7d0ee1a into elastic:main Sep 9, 2025
35 checks passed
@craigtaverner craigtaverner deleted the fix_decay_test-release branch September 9, 2025 18:00
Kubik42 pushed a commit to Kubik42/elasticsearch that referenced this pull request Sep 9, 2025
…SET and INLINESTATS commands) (elastic#134358)

Three PRs have just been merged that cause failures in `test-release`,
and a third PR changes ES|QL railroad diagrams for another recently
released change to `KNN` (also a snapshot-only feature).

* elastic#132729 - `DecayTests` has over 300 failures
* elastic#134029 - `ParsingTests` had six new failures
* elastic#124725 - Failures in `VerifierTests` and `OptimizerVerificationTests`

This PR attempts to fix all in one, because if we do this in separate
PRs they will fail due to the failures cause by the other PRs.
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-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Search Relevance/ES|QL Search functionality in ES|QL Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch >test Issues or PRs that are addressing/adding tests test-release Trigger CI checks against release build v9.2.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants