search

elastic / elasticsearch

Free and Open Source, Distributed, RESTful Search Engine
https://www.elastic.co/products/elasticsearch
Other
69.69k stars 24.66k forks source link

Replace InferenceService.getMinimalSupportedVersion with cluster features or another mechanism #113384

Open thecoop opened 1 week ago

thecoop commented 1 week ago

InferenceService.getMinimalSupportedVersion uses transport version to check for model support, but this is not directly related to the transport protocol. Whilst this works for now, it is misusing the transport version number, as it couples the transport protocol to the specific models being used.

In the future, cluster features or a version number specific to inference models should be used. Existing uses should be refactored to use historical features (we may need to increase the range historical features are applicable for)

elasticsearchmachine commented 1 week ago

Pinging @elastic/ml-core (Team:ML)