Skip to content

Conversation

@szabosteve
Copy link
Contributor

@szabosteve szabosteve commented Sep 23, 2025

Overview

Related issue: https://github.com/elastic/search-team/issues/11055 and #3008

This PR creates a page under the EIS section about when to use EIS and when to use ML nodes.

@github-actions
Copy link

github-actions bot commented Sep 23, 2025

🔍 Preview links for changed docs

@seanhandley seanhandley self-assigned this Sep 23, 2025
@szabosteve szabosteve marked this pull request as ready for review September 23, 2025 15:58
@szabosteve szabosteve requested review from a team as code owners September 23, 2025 15:58
Copy link
Contributor

@florent-leborgne florent-leborgne left a comment

Choose a reason for hiding this comment

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

LGTM. Just a note:

The new page has the hidden attribute, which means that the page won't be published until we remove the hidden attribute.

It will actually be published. But it won't be indexed by search engines, nor appear in the navigation. Meaning it will be accessible with a direct URL only while it's hidden.

@szabosteve
Copy link
Contributor Author

@florent-leborgne Thank you for your note. I updated the PR description accordingly.

@seanhandley seanhandley marked this pull request as draft September 24, 2025 10:58
@szabosteve szabosteve marked this pull request as ready for review September 24, 2025 15:16
@szabosteve szabosteve marked this pull request as draft September 24, 2025 15:17
@shubhaat shubhaat changed the title [E&A] Creates a page for EIS migration guide [E&A] Creates a page for guidance on when to use EIS and when to use ML nodes Sep 24, 2025
@shubhaat
Copy link
Contributor

We should try to not make this ML node v.s. EIS but instead have

  1. Guidance on when to use ML nodes and when to use EIS
  2. If you are using serverless search projects, or ECH and are already using an index using a model running on ML nodes, how do you use the model on EIS? -> the only way to do this is to create a new index, and index the data again.

@szabosteve szabosteve marked this pull request as ready for review November 4, 2025 08:43
@szabosteve szabosteve requested a review from a team as a code owner November 4, 2025 08:43
@seanhandley seanhandley enabled auto-merge (squash) November 5, 2025 19:32
@seanhandley seanhandley merged commit dae99f4 into main Nov 5, 2025
6 of 7 checks passed
@seanhandley seanhandley deleted the szabosteve/eis-support-page branch November 5, 2025 19:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants