-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Add @SuppressWarnings("removal") to not fail the build with -Werror for some JDKs #11923
Conversation
Compatibility status:Checks if related components are compatible with change 9f34e74 Incompatible componentsIncompatible components: [https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/sql.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/performance-analyzer-rca.git] Skipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/k-nn.git] |
❕ Gradle check result for c17acba: UNSTABLE
Please review all flaky tests that succeeded after retry and create an issue if one does not already exist to track the flaky failure. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #11923 +/- ##
============================================
- Coverage 71.44% 71.35% -0.10%
+ Complexity 59397 59386 -11
============================================
Files 4923 4923
Lines 279178 279178
Branches 40581 40581
============================================
- Hits 199470 199214 -256
- Misses 63064 63439 +375
+ Partials 16644 16525 -119 ☔ View full report in Codecov by Sentry. |
…or some JDKs Signed-off-by: Andriy Redko <[email protected]>
…or some JDKs (#11923) (#11930) (cherry picked from commit 411b279) Signed-off-by: Andriy Redko <[email protected]> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
…or some JDKs (opensearch-project#11923) Signed-off-by: Andriy Redko <[email protected]>
…or some JDKs (opensearch-project#11923) Signed-off-by: Andriy Redko <[email protected]>
…or some JDKs (opensearch-project#11923) Signed-off-by: Andriy Redko <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
Description
We are currently building on JDK-21 but with JDK-17 fe the build fails with:
Related Issues
N/A
Check List
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.