Skip to content

ESQL - Fix source loader filter when vectors are excluded#146223

Merged
carlosdelest merged 4 commits intoelastic:mainfrom
carlosdelest:non-issue/esql-source-vectors-exclusion-fix
Apr 20, 2026
Merged

ESQL - Fix source loader filter when vectors are excluded#146223
carlosdelest merged 4 commits intoelastic:mainfrom
carlosdelest:non-issue/esql-source-vectors-exclusion-fix

Conversation

@carlosdelest
Copy link
Copy Markdown
Member

Closes #145799

Fixes a performance issue for source filtering. When dense vector fields are excluded from source, and there are other source paths inclusions, the whole source (except dense vector fields) is loaded instead of using the source inclusion mechanism.

@carlosdelest carlosdelest added >bug Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) auto-backport Automatically create backport pull requests when merged Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch :Search Relevance/ES|QL Search functionality in ES|QL v9.4.0 v9.3.4 v9.5.0 labels Apr 14, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

Hi @carlosdelest, I've created a changelog YAML for you.

@github-actions
Copy link
Copy Markdown
Contributor

🔍 Preview links for changed docs

⏳ Building and deploying preview... View progress

This comment will be updated with preview links when the build is complete.

@github-actions
Copy link
Copy Markdown
Contributor

ℹ️ 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?

@carlosdelest carlosdelest marked this pull request as ready for review April 14, 2026 14:20
@elasticsearchmachine elasticsearchmachine removed the Team:Analytics Meta label for analytical engine team (ESQL/Aggs/Geo) label Apr 14, 2026
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

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

@carlosdelest carlosdelest merged commit df5f86c into elastic:main Apr 20, 2026
35 checks passed
carlosdelest added a commit to carlosdelest/elasticsearch that referenced this pull request Apr 20, 2026
…6223)

* Fix source loader filter when vectors are excluded

* Update docs/changelog/146223.yaml

* [CI] Auto commit changes from spotless

* Update changelog

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
@elasticsearchmachine
Copy link
Copy Markdown
Collaborator

💔 Backport failed

Status Branch Result
9.4
9.3 Commit could not be cherrypicked due to conflicts

You can use sqren/backport to manually backport by running backport --upstream elastic/elasticsearch --pr 146223

elasticsearchmachine pushed a commit that referenced this pull request Apr 20, 2026
…146787)

* Fix source loader filter when vectors are excluded

* Update docs/changelog/146223.yaml

* [CI] Auto commit changes from spotless

* Update changelog

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
@carlosdelest
Copy link
Copy Markdown
Member Author

💚 All backports created successfully

Status Branch Result
9.3

Questions ?

Please refer to the Backport tool documentation

carlosdelest added a commit to carlosdelest/elasticsearch that referenced this pull request Apr 20, 2026
…6223)

* Fix source loader filter when vectors are excluded

* Update docs/changelog/146223.yaml

* [CI] Auto commit changes from spotless

* Update changelog

---------

Co-authored-by: elasticsearchmachine <infra-root+elasticsearchmachine@elastic.co>
(cherry picked from commit df5f86c)

# Conflicts:
#	x-pack/plugin/esql/src/test/java/org/elasticsearch/xpack/esql/planner/EsPhysicalOperationProvidersTests.java
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-backport Automatically create backport pull requests when merged backport pending >bug :Search Relevance/ES|QL Search functionality in ES|QL Team:Search Relevance Meta label for the Search Relevance team in Elasticsearch v9.3.4 v9.4.0 v9.5.0

3 participants