Skip to content
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

Run server with en_US locale #21232

Merged
merged 2 commits into from
Mar 25, 2024
Merged

Run server with en_US locale #21232

merged 2 commits into from
Mar 25, 2024

Conversation

wendigo
Copy link
Contributor

@wendigo wendigo commented Mar 25, 2024

Trino server behavior does generally not depend on locale settings. Use
en_US as this is what Trino is tested with.

Trino server behavior does generally not depend on locale settings.  Use
en_US as this is what Trino is tested with.
@cla-bot cla-bot bot added the cla-signed label Mar 25, 2024
@github-actions github-actions bot added the docs label Mar 25, 2024
@wendigo wendigo force-pushed the serafin/verify-locale branch from 447632b to a85e083 Compare March 25, 2024 11:12
@wendigo wendigo requested review from findepi and dain March 25, 2024 11:26
@wendigo wendigo merged commit 6026f35 into master Mar 25, 2024
106 checks passed
@wendigo wendigo deleted the serafin/verify-locale branch March 25, 2024 17:49
@wendigo
Copy link
Contributor Author

wendigo commented Mar 25, 2024

Thx @dain @findepi

@github-actions github-actions bot added this to the 444 milestone Mar 25, 2024
@mosabua
Copy link
Member

mosabua commented Mar 26, 2024

Do we want release notes since this affects the suggested default JVM config @martint @colebow ?

@findepi
Copy link
Member

findepi commented Mar 27, 2024

yes, good catch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

4 participants