Skip to content
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

[Backport 2.x] Fix weighted shard routing state across search requests #6060

Merged
merged 2 commits into from
Feb 3, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport cd860ec from #6004.

* Fix maintaining state across search requests with weighted shard routing

Signed-off-by: Anshu Agarwal <anshukag@amazon.com>
(cherry picked from commit cd860ec)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@codecov-commenter
Copy link

codecov-commenter commented Jan 29, 2023

Codecov Report

Merging #6060 (cd860ec) into 2.x (579f1a6) will increase coverage by 0.31%.
The diff coverage is 50.13%.

❗ Current head cd860ec differs from pull request most recent head ccf77fa. Consider uploading reports for the commit ccf77fa to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@             Coverage Diff              @@
##                2.x    #6060      +/-   ##
============================================
+ Coverage     70.37%   70.68%   +0.31%     
+ Complexity    58890    58714     -176     
============================================
  Files          4769     4775       +6     
  Lines        282621   280993    -1628     
  Branches      41169    40590     -579     
============================================
- Hits         198899   198626     -273     
+ Misses        67063    66031    -1032     
+ Partials      16659    16336     -323     
Impacted Files Coverage Δ
...ark/time/NanoTimeVsCurrentTimeMillisBenchmark.java 0.00% <0.00%> (ø)
...nsearch/gradle/precommit/LicenseHeadersTask.groovy 0.00% <ø> (ø)
.../java/org/opensearch/gradle/JdkDownloadPlugin.java 36.26% <0.00%> (-1.10%) ⬇️
...rg/opensearch/gradle/OpenSearchTestBasePlugin.java 6.81% <0.00%> (-0.25%) ⬇️
...ternal/InternalDistributionArchiveCheckPlugin.java 0.00% <0.00%> (ø)
...ternal/InternalDistributionArchiveSetupPlugin.java 0.00% <ø> (ø)
...ensearch/gradle/precommit/FilePermissionsTask.java 78.78% <ø> (ø)
...search/gradle/precommit/ForbiddenPatternsTask.java 88.40% <ø> (ø)
...g/opensearch/gradle/precommit/LoggerUsageTask.java 0.00% <ø> (ø)
...ensearch/gradle/precommit/ThirdPartyAuditTask.java 0.00% <0.00%> (ø)
... and 1049 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Feb 2, 2023

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Feb 3, 2023

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testIndexCreateBlockWithAReadOnlyBlock
      1 org.opensearch.cluster.routing.allocation.decider.DiskThresholdDeciderIT.testIndexCreateBlockWhenAllNodesExceededHighWatermark

@andrross andrross merged commit 3d6fe9c into 2.x Feb 3, 2023
@github-actions github-actions bot deleted the backport/backport-6004-to-2.x branch February 3, 2023 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants