-
Notifications
You must be signed in to change notification settings - Fork 25.7k
Consolidates troubleshooting content into the "Returning semantic field embeddings in _source" section #137233
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
kosabogi
merged 22 commits into
elastic:main
from
kosabogi:semantic-blend-troubleshooting-section
Nov 6, 2025
Merged
Changes from 3 commits
Commits
Show all changes
22 commits
Select commit
Hold shift + click to select a range
3ecd9c7
Consolidates the Troubleshooting section
kosabogi 6b49e51
Merge branch 'main' into semantic-blend-troubleshooting-section
kosabogi 7dae585
Merge branch 'main' into semantic-blend-troubleshooting-section
kosabogi fb44113
Update docs/reference/elasticsearch/mapping-reference/semantic-text.md
kosabogi 96d7183
Test applies switch tabs
kosabogi fd7eb83
Adds version specific examples
kosabogi 3767283
Syntax fix
kosabogi d26917b
Fixes syntax
kosabogi 7fbdb71
Merge branch 'main' into semantic-blend-troubleshooting-section
kosabogi fc06c3f
Adds separate tabs for each versions
kosabogi 4db8fb2
Adds only two version tahs
kosabogi e2a1224
Deletes tabs, uses section level applies-to instead
kosabogi cabbfee
Merge branch 'main' into semantic-blend-troubleshooting-section
kosabogi 4ff57b9
Removes response block, adds intros and warning
kosabogi 5be7634
Reorganizes content, adds applies_to where necessary
kosabogi 60b518d
Merge branch 'main' into semantic-blend-troubleshooting-section
kosabogi 037bbf7
Update docs/reference/elasticsearch/mapping-reference/semantic-text.md
kosabogi 576d820
Update docs/reference/elasticsearch/mapping-reference/semantic-text.md
kosabogi 457e7b8
Adds additional information, changes admonition types
kosabogi 511f019
Update docs/reference/elasticsearch/mapping-reference/semantic-text.md
kosabogi a07dcb1
Shorten tab titles
kosabogi 6618e0d
Merge branch 'main' into semantic-blend-troubleshooting-section
kosabogi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please use the version with
exclude_vectorsto false, this is the preferred way to retrieve the field.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for your suggestions, @jimczi!
I deleted the unnecessary "_inference_fields" where we use exclude_vectors.
As Mike suggested in his comment, I created two versions of this: one for 9.0 and 9.1 versions and one for 9.2 since exclude_vectors: false was added only in the 9.2 version.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we start with the preferred and current way of doing things? We can document the old way but it should be a small warning section without needing to repeat the entire response (it's the same).
Uh oh!
There was an error while loading. Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I’ve reordered the sections so that the 9.2 method appears first.
I also added a warning to the 9.0-9.1 method to emphasize that users on 9.2 or later should use the newer approach instead.
I thought it might work better to keep both methods as equivalent sections instead of putting the 9.0-9.1 version in a warning. Since this documentation set covers all versions starting from 9.0, it feels clearer to show both approaches while highlighting which versions they apply to. I labeled the current recommended method as applicable from version 9.2 to make that distinction clear.
Let me know what you think!