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

COMPAT locale provider will be removed in a future release #13934

Closed
wants to merge 15 commits into from

Conversation

akolarkunnu
Copy link
Contributor

@akolarkunnu akolarkunnu commented Jun 3, 2024

Description

From JDK21 onwards a new warning has started to come, "WARNING: COMPAT locale provider will be removed in a future release". So, we have to avoid usage of COMPAT provider. We were setting exlpicitly to COMPAT locale provider in couple of places, this change is to convert COMPAT to CLDR locale provider. After this change, couple of tests started to fail becasue some locale data has minor changes in CLDR compared to COMPAT. For example, day and month short names of GERMAN "de" locale are different in CLDR and COMPAT, just need to add a . in the end for CLDR.

Related Issues

Resolves #11550

Signed-off-by: Abdul Muneer Kolarkunnu muneer.kolarkunnu@netapp.com

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • API changes companion pull request created.
  • Failing checks are inspected and point to the corresponding known issue(s) (See: Troubleshooting Failing Builds)
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)
  • Public documentation issue/PR created

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Description:
From JDK21 onwards a new warning has started to come, "WARNING: COMPAT locale provider will be removed in a future release". So, we have to avoid usage of COMPAT provider.
We were setting exlpicitly to COMPAT locale provider in couple of places, this change is to convert COMPAT to CLDR locale provider.
After this change, couple of tests started to fail becasue some locale data has minor changes in CLDR compared to COMPAT.
For example, day and month short names of GERMAN "de" locale are different in CLDR and COMPAT, just need to add a . in the end for CLDR.

Resolves opensearch-project#11550
Signed-off-by: Abdul Muneer Kolarkunnu <muneer.kolarkunnu@netapp.com>
CHANGELOG.md Outdated Show resolved Hide resolved
Copy link
Contributor

github-actions bot commented Jun 3, 2024

❌ Gradle check result for b4c00a4: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

Copy link
Contributor

github-actions bot commented Jun 3, 2024

❌ Gradle check result for 44a47f3: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@reta
Copy link
Collaborator

reta commented Jun 5, 2024

@akolarkunnu could you please rebase your pull request against latest main (that will fix checks)? thank you

chishui and others added 13 commits June 5, 2024 08:40
* Add comment to XContentBuilder.toString

Signed-off-by: Liyun Xiu <xiliyun@amazon.com>

* Fix typo

Signed-off-by: Liyun Xiu <xiliyun@amazon.com>

---------

Signed-off-by: Liyun Xiu <xiliyun@amazon.com>
…ribution/packages (opensearch-project#13933)

* Bump com.netflix.nebula.ospackage-base in /distribution/packages

Bumps com.netflix.nebula.ospackage-base from 11.9.0 to 11.9.1.

---
updated-dependencies:
- dependency-name: com.netflix.nebula.ospackage-base
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update changelog

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Signed-off-by: Shivansh Arora <hishiv@amazon.com>
…of documents (opensearch-project#13592)

---------

Signed-off-by: Rishabh Maurya <rishabhmaurya05@gmail.com>
…erived field (opensearch-project#13717)


---------

Signed-off-by: Rishabh Maurya <rishabhmaurya05@gmail.com>
)

The Gradle Enterprise plugin has been rebranded as the Develocity
Plugin, resulting in a number of settings being deprecated. This commit
updates the settings per [this migration guide][1] and removes all
warnings from the build.

[1]: https://docs.gradle.com/develocity/gradle-plugin/legacy/#develocity_migration

Signed-off-by: Andrew Ross <andrross@amazon.com>
* Bump actions/checkout from 3 to 4

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <support@github.com>

* Update changelog

Signed-off-by: dependabot[bot] <support@github.com>

---------

Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Peter Nied <petern@amazon.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: dependabot[bot] <dependabot[bot]@users.noreply.github.com>
Co-authored-by: Peter Nied <petern@amazon.com>
Signed-off-by: Prudhvi Godithi <pgodithi@amazon.com>
…#13131)" (opensearch-project#13969)

This reverts commit b9befaa.

Signed-off-by: Andriy Redko <andriy.redko@aiven.io>
…3655)

* Adds cache clear IT

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Cleaned up logic for cache stats ITs

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Adds more tests around tiered spillover cache

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Fixed cache stats behavior for overall /_nodes/stats call

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

cleanup

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Fixed folder structure

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Addressed Sagar's comments

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Addressed Ankit's comments

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Break horrifyingly long test case into many shorter cases

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Added unsupported operation exception to TSC stats holder incrementEvictions()

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

Addressed Sorabh's comments

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

* rerun assemble

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

* rerun gradle

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

* rerun gradle

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

* rerun gradle

Signed-off-by: Peter Alfonsi <petealft@amazon.com>

---------

Signed-off-by: Peter Alfonsi <petealft@amazon.com>
Co-authored-by: Peter Alfonsi <petealft@amazon.com>
Description:
From JDK21 onwards a new warning has started to come, "WARNING: COMPAT locale provider will be removed in a future release". So, we have to avoid usage of COMPAT provider.
We were setting exlpicitly to COMPAT locale provider in couple of places, this change is to convert COMPAT to CLDR locale provider.
After this change, couple of tests started to fail becasue some locale data has minor changes in CLDR compared to COMPAT.
For example, day and month short names of GERMAN "de" locale are different in CLDR and COMPAT, just need to add a . in the end for CLDR.

Resolves opensearch-project#11550
Signed-off-by: Abdul Muneer Kolarkunnu <muneer.kolarkunnu@netapp.com>
Description:
From JDK21 onwards a new warning has started to come, "WARNING: COMPAT locale provider will be removed in a future release". So, we have to avoid usage of COMPAT provider.
We were setting exlpicitly to COMPAT locale provider in couple of places, this change is to convert COMPAT to CLDR locale provider.
After this change, couple of tests started to fail becasue some locale data has minor changes in CLDR compared to COMPAT.
For example, day and month short names of GERMAN "de" locale are different in CLDR and COMPAT, just need to add a . in the end for CLDR.

Resolves opensearch-project#11550
Signed-off-by: Abdul Muneer Kolarkunnu muneer.kolarkunnu@netapp.com
Copy link
Contributor

github-actions bot commented Jun 5, 2024

❌ Gradle check result for d44b68a: FAILURE

Please examine the workflow log, locate, and copy-paste the failure(s) below, then iterate to green. Is the failure a flaky test unrelated to your change?

@akolarkunnu
Copy link
Contributor Author

@akolarkunnu could you please rebase your pull request against latest main (that will fix checks)? thank you

Closed this PR and created a fresh clean PR - #13988

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Build Build Tasks/Gradle Plugin, groovy scripts, build tools, Javadoc enforcement. CI CI related good first issue Good for newcomers Plugins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

COMPAT locale provider will be removed in a future release
9 participants