Skip to content

Document multi index query support for simplified retrievers #134980

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Sep 18, 2025
Merged

Document multi index query support for simplified retrievers #134980

merged 4 commits into from
Sep 18, 2025

Conversation

ioanatia
Copy link
Contributor

Also addressed a test change from #134822 (comment)

We removed the limitation for being to able a single index with the simplified RRF and linear retrievers in:

#133720
#134822

so we can remove this limitation from the documentation.

@ioanatia ioanatia added >docs General docs changes :SearchOrg/Relevance Label for the Search (solution/org) Relevance team Team:Search - Relevance The Search organization Search Relevance team v9.2.0 labels Sep 18, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 18, 2025 •
edited
Loading

🔍 Preview links for changed docs

@github-actions GitHub Actions
Copy link
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?

ioanatia
ioanatia commented Sep 18, 2025
View reviewed changes
docs/reference/elasticsearch/rest-apis/retrievers.md Outdated
### Limitations

- **Single index**: Multi-field queries currently work with single index searches only
- **Single index**: Applies only to the 9.1 version: multi-field queries work with single index searches only.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

these are the docs we added for the simplified retriever #130559

And this is the only mention I see of this limitation.
I wanted to see if I could use applies_to here but I couldn't find an example where we use it when we remove a limitation.
cc @leemthompo

leemthompo
leemthompo reviewed Sep 18, 2025
View reviewed changes
@ioanatia ioanatia marked this pull request as ready for review September 18, 2025 11:30
@elasticsearchmachine elasticsearchmachine added the Team:Docs Meta label for docs team label Sep 18, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/core-docs (Team:Docs)

@ioanatia ioanatia requested a review from leemthompo September 18, 2025 12:02
kderusso
kderusso approved these changes Sep 18, 2025
View reviewed changes
docs/reference/elasticsearch/rest-apis/retrievers.md
### Limitations

- **Single index**: Multi-field queries currently work with single index searches only
- **Single index**: Until 9.2, multi-field queries only work with single index searches.
Copy link
Member

Choose a reason for hiding this comment

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

Looks good, I think the applies to syntax could be a bit heavy handed here but will defer to @leemthompo for best practices 😁

Copy link
Contributor

Choose a reason for hiding this comment

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

yeah applies_to doesn't work in every scenario :)

@ioanatia ioanatia merged commit 75ed968 into elastic:main Sep 18, 2025
34 checks passed
@ioanatia ioanatia deleted the simplified_retrievers_docs branch September 18, 2025 15:01
gmjehovich pushed a commit to gmjehovich/elasticsearch that referenced this pull request Sep 18, 2025
@ioanatia @gmjehovich
Document multi index query support for simplified retrievers (elastic…
a160a49 
…#134980)
szybia added a commit to szybia/elasticsearch that referenced this pull request Sep 18, 2025
@szybia
Merge remote-tracking branch 'upstream/main' into dlm-tiny-retentions
23d05ac 
* upstream/main: (43 commits)
  Unmute testAckedIndexing to see if it still fails on main (elastic#134682)
  Silence time zone ID deprecation warning for JDK 25 due to log4j2 bug. (elastic#134719)
  Adding a getUnmodifiableSourceAndMetadata() method to IngestDocument (elastic#134816)
  Mark the create-index-from-source action as publicly available on Serverless (elastic#134953)
  ESQL: Rename command from INLINESTATS to INLINE STATS (elastic#134827)
  Document multi index query support for simplified retrievers (elastic#134980)
  [ML] Fix YAMl test to use correct query parameter type (elastic#134999)
  [Transform] Wait for PIT to close (elastic#134955)
  Add XPath to XmlUtils (elastic#134923)
  Fixing conditional processor mutability bugs (elastic#134936)
  [Transform] Lower loglevel of 3 transform-related error messages from ERROR to WARN (elastic#134985)
  Unmute pattern text tests. (elastic#134981)
  Integrate weights into simplified RRF retriever syntax (elastic#132680)
  Mute org.elasticsearch.xpack.esql.qa.mixed.MixedClusterEsqlSpecIT test {csv-spec:stats.CountDistinctWithConditions} elastic#134993
  Update periodic java-ea build to test java 26 pre-release (elastic#134983)
  Mute org.elasticsearch.xpack.esql.ccq.MultiClusterSpecIT test {csv-spec:stats.CountDistinctWithConditions} elastic#134984
  Fix and unmute testIndexSettingProviderPrivateSetting (elastic#134861)
  Add missing common cat params (elastic#134870)
  Support querying multiple indices with the simplified RRF retriever (elastic#134822)
  Allow including semantic field embeddings in _source (elastic#134717)
  ...
phananh1010 added a commit to phananh1010/elasticsearch that referenced this pull request Sep 23, 2025
@phananh1010
Mirror upstream elastic#134980 as single snapshot commit for AI review
423182a 
BASE=eb3bc2f2929209461802884ea01f77d3b3fcf96a
HEAD=16aecaca5333692fff65a3d71818c0d6de287e37
Branch=main
phananh1010 added a commit to phananh1010/elasticsearch that referenced this pull request Oct 2, 2025
@phananh1010
Mirror upstream elastic#134980 as single snapshot commit for AI review
61167f5 
BASE=eb3bc2f2929209461802884ea01f77d3b3fcf96a
HEAD=16aecaca5333692fff65a3d71818c0d6de287e37
Branch=main
phananh1010 added a commit to phananh1010/elasticsearch that referenced this pull request Oct 6, 2025
@phananh1010
Mirror upstream elastic#134980 as single snapshot commit for AI review
e47cc8c 
BASE=eb3bc2f2929209461802884ea01f77d3b3fcf96a
HEAD=16aecaca5333692fff65a3d71818c0d6de287e37
Branch=main
@coderabbitai coderabbitai bot mentioned this pull request Oct 6, 2025
Closed
phananh1010 added a commit to phananh1010/elasticsearch that referenced this pull request Oct 17, 2025
@phananh1010
Mirror upstream elastic#134980 as single snapshot commit for AI review
05d1573 
BASE=eb3bc2f2929209461802884ea01f77d3b3fcf96a
HEAD=16aecaca5333692fff65a3d71818c0d6de287e37
Branch=main
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@leemthompo leemthompo leemthompo approved these changes

@kderusso kderusso kderusso approved these changes

Assignees

No one assigned

Labels

>docs General docs changes :SearchOrg/Relevance Label for the Search (solution/org) Relevance team Team:Docs Meta label for docs team Team:Search - Relevance The Search organization Search Relevance team v9.2.0

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@ioanatia @elasticsearchmachine @leemthompo @kderusso