Skip to content

Conversation

davidkyle
Copy link
Member

Cast the total_search_time_ms to a double when sorting as described in https://www.elastic.co/docs/reference/elasticsearch/rest-apis/sort-search-results#_sorting_numeric_fields. This guards against problems where the field has different mappings.

@davidkyle davidkyle added >bug :ml Machine learning v9.2.0 labels Sep 19, 2025
@elasticsearchmachine elasticsearchmachine added the Team:ML Meta label for the ML team label Sep 19, 2025
@elasticsearchmachine
Copy link
Collaborator

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

@elasticsearchmachine
Copy link
Collaborator

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

@davidkyle davidkyle added v8.19.5 v9.1.5 auto-backport Automatically create backport pull requests when merged labels Sep 19, 2025
Copy link
Contributor

@przemekwitek przemekwitek left a comment

Choose a reason for hiding this comment

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

LGTM

@davidkyle davidkyle enabled auto-merge (squash) September 24, 2025 14:37
@davidkyle davidkyle merged commit 62c6c1a into elastic:main Sep 24, 2025
33 of 34 checks passed
davidkyle added a commit to davidkyle/elasticsearch that referenced this pull request Sep 24, 2025
@elasticsearchmachine
Copy link
Collaborator

💚 Backport successful

Status Branch Result
8.19
9.1

davidkyle added a commit to davidkyle/elasticsearch that referenced this pull request Sep 24, 2025
elasticsearchmachine pushed a commit that referenced this pull request Sep 24, 2025
Co-authored-by: Pat Whelan <pat.whelan@elastic.co>
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 :ml Machine learning Team:ML Meta label for the ML team v8.19.5 v9.1.5 v9.2.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants