-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Replace the right version to fix backward compatibility introduced wi… #13168
Conversation
…th DerivedFields feature Signed-off-by: Rishabh Maurya <[email protected]>
Compatibility status:Checks if related components are compatible with change 3c6dbf0 Incompatible componentsSkipped componentsCompatible componentsCompatible components: [https://github.com/opensearch-project/custom-codecs.git, https://github.com/opensearch-project/asynchronous-search.git, https://github.com/opensearch-project/performance-analyzer-rca.git, https://github.com/opensearch-project/flow-framework.git, https://github.com/opensearch-project/cross-cluster-replication.git, https://github.com/opensearch-project/job-scheduler.git, https://github.com/opensearch-project/reporting.git, https://github.com/opensearch-project/security.git, https://github.com/opensearch-project/opensearch-oci-object-storage.git, https://github.com/opensearch-project/geospatial.git, https://github.com/opensearch-project/common-utils.git, https://github.com/opensearch-project/k-nn.git, https://github.com/opensearch-project/neural-search.git, https://github.com/opensearch-project/anomaly-detection.git, https://github.com/opensearch-project/security-analytics.git, https://github.com/opensearch-project/performance-analyzer.git, https://github.com/opensearch-project/ml-commons.git, https://github.com/opensearch-project/notifications.git, https://github.com/opensearch-project/index-management.git, https://github.com/opensearch-project/observability.git, https://github.com/opensearch-project/alerting.git, https://github.com/opensearch-project/sql.git] |
❌ Gradle check result for 3c6dbf0: 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? |
I just merged #13163 and retried the Gradle check here |
❌ Gradle check result for 3c6dbf0: 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? |
Looks like a node connection exception. Will try check again |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #13168 +/- ##
============================================
+ Coverage 71.42% 71.51% +0.09%
- Complexity 59978 60566 +588
============================================
Files 4985 5033 +48
Lines 282275 284903 +2628
Branches 40946 41286 +340
============================================
+ Hits 201603 203745 +2142
- Misses 63999 64204 +205
- Partials 16673 16954 +281 ☔ View full report in Codecov by Sentry. |
…th DerivedFields feature (opensearch-project#13168) Signed-off-by: Rishabh Maurya <[email protected]> Signed-off-by: Shivansh Arora <[email protected]>
…th DerivedFields feature
Description
Fixes backward compatibility issues which was introduced in #12850
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
Check List
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)[ ] Public documentation issue/PR createdBy 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.