Remove LegacyESVersion#19793
Conversation
|
❌ Gradle check result for 3352c92: null 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? |
3352c92 to
31abbd4
Compare
|
❌ Gradle check result for 31abbd4: 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? |
31abbd4 to
74ec295
Compare
|
❌ Gradle check result for 74ec295: 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? |
124a2f3 to
8cd2c55
Compare
|
❌ Gradle check result for 8cd2c55: 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? |
|
❌ Gradle check result for fa4ef0c: 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? |
|
❌ Gradle check result for 5b18875: 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? |
|
❕ Gradle check result for f3afd9f: 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 Report❌ Patch coverage is Additional details and impacted files@@ Coverage Diff @@
## main #19793 +/- ##
============================================
- Coverage 73.21% 73.13% -0.09%
+ Complexity 71254 71137 -117
============================================
Files 5766 5766
Lines 325470 325362 -108
Branches 47084 47053 -31
============================================
- Hits 238296 237940 -356
- Misses 68043 68296 +253
+ Partials 19131 19126 -5 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
❌ Gradle check result for e91892b: 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? |
e91892b to
1f28669
Compare
|
❌ Gradle check result for 1f28669: 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? |
|
❕ Gradle check result for 1f28669: 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. |
Since OpenSearch 3.0, we no longer try to maintain backward compatibility with Elasticsearch versions. The LegacyESVersion class is not needed anymore. Signed-off-by: Michael Froh <msfroh@apache.org>
Going forward, any call to `Version.fromId` should pass an OpenSearch ID, so we should check for the OpenSearch mask. Signed-off-by: Michael Froh <msfroh@apache.org>
Signed-off-by: Michael Froh <msfroh@apache.org>
Signed-off-by: Michael Froh <msfroh@apache.org>
These tests previously relied on strings with parts going up to 999, which was never properly supported (since they would overflow into the next part). Signed-off-by: Michael Froh <msfroh@apache.org>
Signed-off-by: Michael Froh <msfroh@apache.org>
We need to support handshakes 2 major versions back. For some reason, this is considered to be 7.9, which is no longer a supported version. Signed-off-by: Michael Froh <msfroh@apache.org>
Make japicmp happy. Signed-off-by: Michael Froh <msfroh@apache.org>
Signed-off-by: Michael Froh <msfroh@apache.org>
Signed-off-by: Michael Froh <msfroh@apache.org>
1f28669 to
4fa0b7d
Compare
|
❌ Gradle check result for 4fa0b7d: 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? |
Description
Since OpenSearch 3.0, we no longer try to maintain backward compatibility with Elasticsearch versions. The LegacyESVersion class is not needed anymore.
Related Issues
Resolves #18387
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.