Skip to content

Conversation

nik9000
Copy link
Member

@nik9000 nik9000 commented May 6, 2025

Make TO_LOWER and TO_UPPER process all values it received.

This is quite large because it borrows a lot of code from the regular evaluator generator to generate conversions so we can use the Locale. That change propagates to the order of some parameters and to the toString and a few more places.

Closes #124002

Make `TO_LOWER` and `TO_UPPER` process all values it received.

This is quite large because it borrows a lot of code from the regular
evaluator generator to generate conversions so we can use the Locale.
That change propagates to the order of some parameters and to the
`toString` and a few more places.

Closes elastic#124002
Copy link
Contributor

github-actions bot commented May 6, 2025

Documentation preview:

@nik9000 nik9000 added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label May 6, 2025
@elasticsearchmachine elasticsearchmachine merged commit 9633d6c into elastic:8.19 May 8, 2025
15 checks passed
@nik9000 nik9000 deleted the to_lower_mv_8_19 branch May 8, 2025 14:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
:Analytics/ES|QL AKA ESQL auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) backport >bug v8.19.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants