Skip to content
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

Always Emit Inference ID in Semantic Text Mapping #117294

Merged
merged 5 commits into from
Nov 22, 2024

Conversation

Mikep86
Copy link
Contributor

@Mikep86 Mikep86 commented Nov 21, 2024

Always emit the inference_id param when writing a semantic_text field mapping to XContent.

@Mikep86 Mikep86 added >bug :Search Foundations/Mapping Index mappings, including merging and defining field types :SearchOrg/Relevance Label for the Search (solution/org) Relevance team v9.0.0 v8.17.1 v8.18.0 labels Nov 21, 2024
@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/search-eng (Team:SearchOrg)

@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

Pinging @elastic/es-search-foundations (Team:Search Foundations)

@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Contributor

@jimczi jimczi left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@kderusso kderusso left a comment

Choose a reason for hiding this comment

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

Changes LGTM but I'd like to request two additional things:

  1. A Yaml test that shows the default mapping
  2. Documentation update that says the inference ID is always returned (that can come in a followup due to time)
    Thanks for addressing this so quickly.

@Mikep86
Copy link
Contributor Author

Mikep86 commented Nov 21, 2024

@elasticmachine update branch

Copy link
Member

@kderusso kderusso left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you!

@Mikep86 Mikep86 added the auto-backport Automatically create backport pull requests when merged label Nov 22, 2024
@Mikep86 Mikep86 merged commit 830c504 into elastic:main Nov 22, 2024
16 checks passed
Mikep86 added a commit to Mikep86/elasticsearch that referenced this pull request Nov 22, 2024
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.17
8.x

Mikep86 added a commit to Mikep86/elasticsearch that referenced this pull request Nov 22, 2024
smalyshev pushed a commit to smalyshev/elasticsearch that referenced this pull request Nov 22, 2024
Samiul-TheSoccerFan added a commit to elastic/kibana that referenced this pull request Nov 26, 2024
## Summary

The Index mapping will have access to default elser inference endpoint
so we do not need to hardcode endpoint names in the Kibana.

This needs to go after
#elastic/elasticsearch#117294 merges in `8.17`
and further.


https://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd

---------

Co-authored-by: Elastic Machine <[email protected]>
Samiul-TheSoccerFan added a commit to Samiul-TheSoccerFan/kibana that referenced this pull request Nov 26, 2024
## Summary

The Index mapping will have access to default elser inference endpoint
so we do not need to hardcode endpoint names in the Kibana.

This needs to go after
#elastic/elasticsearch#117294 merges in `8.17`
and further.

https://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit b3d638b)

# Conflicts:
#	x-pack/plugins/translations/translations/zh-CN.json
Samiul-TheSoccerFan added a commit to Samiul-TheSoccerFan/kibana that referenced this pull request Nov 26, 2024
## Summary

The Index mapping will have access to default elser inference endpoint
so we do not need to hardcode endpoint names in the Kibana.

This needs to go after
#elastic/elasticsearch#117294 merges in `8.17`
and further.

https://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd

---------

Co-authored-by: Elastic Machine <[email protected]>
(cherry picked from commit b3d638b)

# Conflicts:
#	x-pack/plugins/translations/translations/zh-CN.json
Samiul-TheSoccerFan added a commit to elastic/kibana that referenced this pull request Nov 26, 2024
# Backport

This will backport the following commits from `main` to `8.x`:
- [Remove hardcoded preconfigured ELSER endpoint
(#201300)](#201300)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Samiul
Monir","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-26T14:42:20Z","message":"Remove
hardcoded preconfigured ELSER endpoint (#201300)\n\n##
Summary\r\n\r\nThe Index mapping will have access to default elser
inference endpoint\r\nso we do not need to hardcode endpoint names in
the Kibana.\r\n\r\nThis needs to go
after\r\n#elastic/elasticsearch#117294 merges in
`8.17`\r\nand
further.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"b3d638b7cf3022fbe1e0fb019462ad9df6d52f25","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:version","v8.17.0","v8.18.0"],"number":201300,"url":"https://github.com/elastic/kibana/pull/201300","mergeCommit":{"message":"Remove
hardcoded preconfigured ELSER endpoint (#201300)\n\n##
Summary\r\n\r\nThe Index mapping will have access to default elser
inference endpoint\r\nso we do not need to hardcode endpoint names in
the Kibana.\r\n\r\nThis needs to go
after\r\n#elastic/elasticsearch#117294 merges in
`8.17`\r\nand
further.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"b3d638b7cf3022fbe1e0fb019462ad9df6d52f25"}},"sourceBranch":"main","suggestedTargetBranches":["8.17","8.x"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201300","number":201300,"mergeCommit":{"message":"Remove
hardcoded preconfigured ELSER endpoint (#201300)\n\n##
Summary\r\n\r\nThe Index mapping will have access to default elser
inference endpoint\r\nso we do not need to hardcode endpoint names in
the Kibana.\r\n\r\nThis needs to go
after\r\n#elastic/elasticsearch#117294 merges in
`8.17`\r\nand
further.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"b3d638b7cf3022fbe1e0fb019462ad9df6d52f25"}},{"branch":"8.17","label":"v8.17.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","labelRegex":"^v8.18.0$","isSourceBranch":false,"state":"NOT_CREATED"}]}]
BACKPORT-->
Samiul-TheSoccerFan added a commit to elastic/kibana that referenced this pull request Nov 26, 2024
# Backport

This will backport the following commits from `main` to `8.17`:
- [Remove hardcoded preconfigured ELSER endpoint
(#201300)](#201300)

<!--- Backport version: 8.9.8 -->

### Questions ?
Please refer to the [Backport tool
documentation](https://github.com/sqren/backport)

<!--BACKPORT [{"author":{"name":"Samiul
Monir","email":"[email protected]"},"sourceCommit":{"committedDate":"2024-11-26T14:42:20Z","message":"Remove
hardcoded preconfigured ELSER endpoint (#201300)\n\n##
Summary\r\n\r\nThe Index mapping will have access to default elser
inference endpoint\r\nso we do not need to hardcode endpoint names in
the Kibana.\r\n\r\nThis needs to go
after\r\n#elastic/elasticsearch#117294 merges in
`8.17`\r\nand
further.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"b3d638b7cf3022fbe1e0fb019462ad9df6d52f25","branchLabelMapping":{"^v9.0.0$":"main","^v8.18.0$":"8.x","^v(\\d+).(\\d+).\\d+$":"$1.$2"}},"sourcePullRequest":{"labels":["release_note:skip","v9.0.0","Team:Search","backport:version","v8.17.0","v8.18.0"],"number":201300,"url":"https://github.com/elastic/kibana/pull/201300","mergeCommit":{"message":"Remove
hardcoded preconfigured ELSER endpoint (#201300)\n\n##
Summary\r\n\r\nThe Index mapping will have access to default elser
inference endpoint\r\nso we do not need to hardcode endpoint names in
the Kibana.\r\n\r\nThis needs to go
after\r\n#elastic/elasticsearch#117294 merges in
`8.17`\r\nand
further.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"b3d638b7cf3022fbe1e0fb019462ad9df6d52f25"}},"sourceBranch":"main","suggestedTargetBranches":["8.17"],"targetPullRequestStates":[{"branch":"main","label":"v9.0.0","labelRegex":"^v9.0.0$","isSourceBranch":true,"state":"MERGED","url":"https://github.com/elastic/kibana/pull/201300","number":201300,"mergeCommit":{"message":"Remove
hardcoded preconfigured ELSER endpoint (#201300)\n\n##
Summary\r\n\r\nThe Index mapping will have access to default elser
inference endpoint\r\nso we do not need to hardcode endpoint names in
the Kibana.\r\n\r\nThis needs to go
after\r\n#elastic/elasticsearch#117294 merges in
`8.17`\r\nand
further.\r\n\r\n\r\nhttps://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd\r\n\r\n---------\r\n\r\nCo-authored-by:
Elastic Machine
<[email protected]>","sha":"b3d638b7cf3022fbe1e0fb019462ad9df6d52f25"}},{"branch":"8.17","label":"v8.17.0","labelRegex":"^v(\\d+).(\\d+).\\d+$","isSourceBranch":false,"state":"NOT_CREATED"},{"branch":"8.x","label":"v8.18.0","labelRegex":"^v8.18.0$","isSourceBranch":false,"url":"https://github.com/elastic/kibana/pull/201811","number":201811,"state":"OPEN"}]}]
BACKPORT-->
paulinashakirova pushed a commit to paulinashakirova/kibana that referenced this pull request Nov 26, 2024
## Summary

The Index mapping will have access to default elser inference endpoint
so we do not need to hardcode endpoint names in the Kibana.

This needs to go after
#elastic/elasticsearch#117294 merges in `8.17`
and further.


https://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd

---------

Co-authored-by: Elastic Machine <[email protected]>
alexey-ivanov-es pushed a commit to alexey-ivanov-es/elasticsearch that referenced this pull request Nov 28, 2024
CAWilson94 pushed a commit to CAWilson94/kibana that referenced this pull request Dec 12, 2024
## Summary

The Index mapping will have access to default elser inference endpoint
so we do not need to hardcode endpoint names in the Kibana.

This needs to go after
#elastic/elasticsearch#117294 merges in `8.17`
and further.


https://github.com/user-attachments/assets/4a786fde-e250-440d-a9d7-2256dacc8edd

---------

Co-authored-by: Elastic Machine <[email protected]>
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 >bug :Search Foundations/Mapping Index mappings, including merging and defining field types :SearchOrg/Relevance Label for the Search (solution/org) Relevance team v8.17.1 v8.18.0 v9.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants