-
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
Ignoring unavailable shards during search request execution with igno… #13298
Conversation
…re_available parameter Signed-off-by: Ankit Jain <akjain@amazon.com>
❌ Gradle check result for 4cdb973: 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 3f556a8: 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? |
Failing tests:
|
Looks good! I'm going to keep retrying Gradle checks until it passes. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #13298 +/- ##
============================================
- Coverage 71.42% 71.40% -0.02%
- Complexity 59978 60646 +668
============================================
Files 4985 5039 +54
Lines 282275 285433 +3158
Branches 40946 41338 +392
============================================
+ Hits 201603 203805 +2202
- Misses 63999 64807 +808
- Partials 16673 16821 +148 ☔ View full report in Codecov by Sentry. |
@msfroh - It seems the gradle check passed! |
…re_available parameter
Description
Better approach compared to #13209
Related Issues
Resolves #12371
Check List
- [ ] 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.