Skip to content

[8.19] [EIS] Dense Text Embedding task type integration (#129847) #129963

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
merged 6 commits into from
Jun 25, 2025

Conversation

timgrein
Copy link
Contributor

Backport

This will backport the following commits from main to 8.19:

Questions ?

Please refer to the Backport tool documentation

(cherry picked from commit 3b51dd5)

# Conflicts:
#	server/src/main/java/org/elasticsearch/TransportVersions.java
#	x-pack/plugin/inference/qa/inference-service-tests/src/javaRestTest/java/org/elasticsearch/xpack/inference/InferenceGetModelsWithElasticInferenceServiceIT.java
#	x-pack/plugin/inference/qa/inference-service-tests/src/javaRestTest/java/org/elasticsearch/xpack/inference/MockElasticInferenceServiceAuthorizationServer.java
#	x-pack/plugin/inference/src/internalClusterTest/java/org/elasticsearch/xpack/inference/integration/InferenceRevokeDefaultEndpointsIT.java
#	x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/ElasticInferenceService.java
#	x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/action/ElasticInferenceServiceActionVisitor.java
#	x-pack/plugin/inference/src/main/java/org/elasticsearch/xpack/inference/services/elastic/response/ElasticInferenceServiceAuthorizationResponseEntity.java
#	x-pack/plugin/inference/src/test/java/org/elasticsearch/xpack/inference/services/elastic/ElasticInferenceServiceTests.java
@timgrein timgrein added :ml Machine learning Team:ML Meta label for the ML team labels Jun 24, 2025
@brendan-jugan-elastic
Copy link
Contributor

@timgrein took a look at this and wondering if I need to backport my default inference endpoint changes for rerank on EIS?

I noticed this difference between this PR and your initial PR

@timgrein
Copy link
Contributor Author

@timgrein took a look at this and wondering if I need to backport my default inference endpoint changes for rerank on EIS?

I noticed this difference between this PR and your initial PR

Yes, we also want rerank in 8.19 (9.1 and 8.19 should be en par AFAIU)

@brendan-jugan-elastic brendan-jugan-elastic self-requested a review June 25, 2025 13:22
@timgrein timgrein merged commit 7d09f05 into elastic:8.19 Jun 25, 2025
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport :ml Machine learning Team:ML Meta label for the ML team v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants