-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Switch to ShardsBatchGatewayAllocator in batch enabled mode for shard… #14107
Switch to ShardsBatchGatewayAllocator in batch enabled mode for shard… #14107
Conversation
❌ Gradle check result for 27a1e00: 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? |
server/src/main/java/org/opensearch/cluster/routing/allocation/AllocationService.java
Outdated
Show resolved
Hide resolved
server/src/main/java/org/opensearch/cluster/routing/allocation/AllocationService.java
Show resolved
Hide resolved
… explain allocation API Signed-off-by: Swetha Guptha <gupthasg@amazon.com>
27a1e00
to
385dbe6
Compare
❌ Gradle check result for 385dbe6: 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? |
Failures:
Flaky test: |
❌ Gradle check result for 385dbe6: 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 385dbe6: 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 385dbe6: 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? |
Latest gradle run failure:
flaky test: #14173 |
❌ Gradle check result for 385dbe6: 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 385dbe6: 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 385dbe6: 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? |
|
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #14107 +/- ##
============================================
+ Coverage 71.42% 71.71% +0.29%
- Complexity 59978 61429 +1451
============================================
Files 4985 5073 +88
Lines 282275 288527 +6252
Branches 40946 41777 +831
============================================
+ Hits 201603 206923 +5320
- Misses 63999 64464 +465
- Partials 16673 17140 +467 ☔ View full report in Codecov by Sentry. |
2e13e9c
into
opensearch-project:main
… explain allocation API (#14107) (#14218) (cherry picked from commit 2e13e9c) Signed-off-by: Swetha Guptha <gupthasg@amazon.com> 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> Co-authored-by: Swetha Guptha <gupthasg@amazon.com>
… explain allocation API (#14107) (#14340) (cherry picked from commit 2e13e9c) Signed-off-by: Swetha Guptha <gupthasg@amazon.com> 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> Co-authored-by: Swetha Guptha <gupthasg@amazon.com>
… explain allocation API (opensearch-project#14107) Signed-off-by: Swetha Guptha <gupthasg@amazon.com> Co-authored-by: Swetha Guptha <gupthasg@amazon.com>
… explain allocation API (opensearch-project#14107) (opensearch-project#14218) (cherry picked from commit 2e13e9c) Signed-off-by: Swetha Guptha <gupthasg@amazon.com> 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> Co-authored-by: Swetha Guptha <gupthasg@amazon.com> Signed-off-by: kkewwei <kkewwei@163.com>
… explain allocation API (opensearch-project#14107) Signed-off-by: Swetha Guptha <gupthasg@amazon.com> Co-authored-by: Swetha Guptha <gupthasg@amazon.com>
Description
Change in AllocationService to switch to ShardsBatchGatewayAllocator for serving API _cluster/allocation/explain when batch mode is enabled.
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
#13990
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.