Skip to content

Rewritten integer sorts need to use SortedNumericSortField (#139538)#139700

Merged
elasticsearchmachine merged 1 commit intoelastic:patch/serverless-fixfrom
romseygeek:patch/serverless-fix-sorting
Dec 17, 2025
Merged

Rewritten integer sorts need to use SortedNumericSortField (#139538)#139700
elasticsearchmachine merged 1 commit intoelastic:patch/serverless-fixfrom
romseygeek:patch/serverless-fix-sorting

Conversation

@romseygeek
Copy link
Contributor

Sorts against integer fields in pre-8.19 indexes are rewritten to use
LongComparators to stay consistent with pre-written index metadata.
These also need to always expect multi-valued fields. #139293 incorrectly
built single-valued sorts in certain circumstances, which would then
throw runtime errors.

…39538)

Sorts against integer fields in pre-8.19 indexes are rewritten to use
LongComparators to stay consistent with pre-written index metadata.
These also need to always expect multi-valued fields.  elastic#139293
incorrectly built single-valued sorts in certain circumstances, which
would then throw runtime errors.
@romseygeek romseygeek requested a review from martijnvg December 17, 2025 15:33
@romseygeek romseygeek self-assigned this Dec 17, 2025
@romseygeek romseygeek added >non-issue :Search/Search Search-related issues that do not fall into other categories v9.3.0 labels Dec 17, 2025
@elasticsearchmachine elasticsearchmachine added serverless-linked Added by automation, don't add manually Team:Search Meta label for search team labels Dec 17, 2025
@elasticsearchmachine
Copy link
Collaborator

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

Copy link
Member

@martijnvg martijnvg left a comment

Choose a reason for hiding this comment

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

👍

@romseygeek romseygeek added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Dec 17, 2025
@elasticsearchmachine elasticsearchmachine merged commit 0a4423b into elastic:patch/serverless-fix Dec 17, 2025
36 checks passed
@romseygeek romseygeek deleted the patch/serverless-fix-sorting branch December 17, 2025 17:19
szybia added a commit to szybia/elasticsearch that referenced this pull request Jan 7, 2026
* upstream/main:
  Add hook for blocking termination (elastic#133555)
  Delegate to ES93ScalarQuantizedVectorsFormat rather than copying behaviour (elastic#139834)
  Refactor compound block types (elastic#140219)
  Flush the rate buffer when the slice index changes (elastic#138856)
  Log linked project connection errors at debug during shutdown (elastic#140239)
  Periodic FIPS 140-3 buildkite pipelines (elastic#139909)
  ES|QL - Remove TERM function (elastic#139953)
  Fix name of started time field in shutdown status (elastic#139910)
  Drop `project_routing` from query params (elastic#140272)
  Fix flaky test: AllocationDecidersTests (elastic#140271)
  Add List Reindex API (elastic#140184)
  Expose _tier metadata attribute in ESQL (elastic#139894)
  Tweak TSDBRestEsqlIT#testTimeSeriesQuerying(...) (elastic#140210)
  Fix an OOM error when creating to many chained synonym graph token filter. (elastic#140026)
  Suppress Azure SDK error logs (elastic#139730)
  Rewritten integer sorts need to use SortedNumericSortField (elastic#139538) (elastic#139700)
  Adjust index versions for skippers for time series (elastic#139670)
  Fix host.name skippers index version range (elastic#139636)
  Remove BWC shim for a broken commit
  Fix index.mapping.use_doc_values_skipper defaults in serverless (elastic#139532)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) >non-issue :Search/Search Search-related issues that do not fall into other categories serverless-linked Added by automation, don't add manually Team:Search Meta label for search team v9.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments