[Backport] 17295 to 2.x #17464
Open
[Backport] 17295 to 2.x #17464
+1,326
−28
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…hards per node and per index (opensearch-project#17295) * Added a new index level setting to limit the total primary shards per index per node. Added relevant files for unit test and integration test. Signed-off-by: Divyansh Pandey <[email protected]> * update files for code quality Signed-off-by: Divyansh Pandey <[email protected]> * moved primary shard count function to RoutingNode.java Signed-off-by: Divyansh Pandey <[email protected]> * removed unwanted files Signed-off-by: Divyansh Pandey <[email protected]> * added cluster level setting to limit total primary shards per node Signed-off-by: Divyansh Pandey <[email protected]> * allow the index level settings to be applied to both DOCUMENT and SEGMENT replication indices Signed-off-by: Divyansh Pandey <[email protected]> * Added necessary validator to restrict the index and cluster level primary shards per node settings only for remote store enabled cluster. Added relevant unit and integration tests. Signed-off-by: Divyansh Pandey <[email protected]> * refactoring changes Signed-off-by: Divyansh Pandey <[email protected]> * refactoring changes Signed-off-by: Divyansh Pandey <[email protected]> * Empty commit to rerun gradle test Signed-off-by: Divyansh Pandey <[email protected]> * optimised the calculation of total primary shards on a node Signed-off-by: Divyansh Pandey <[email protected]> * Refactoring changes Signed-off-by: Divyansh Pandey <[email protected]> * refactoring changes, added TODO to MetadataCreateIndexService Signed-off-by: Divyansh Pandey <[email protected]> * Added integration test for scenario where primary shards setting is set for cluster which is not remote store enabled Signed-off-by: Divyansh Pandey <[email protected]> --------- Signed-off-by: Divyansh Pandey <[email protected]> Signed-off-by: Divyansh Pandey <[email protected]> Co-authored-by: Divyansh Pandey <[email protected]> (cherry picked from commit bc209ee)
pandeydivyansh1803
requested review from
anasalkouz,
andrross,
ashking94,
Bukhtawar,
CEHENKLE,
cwperks,
dblock,
dbwiddis,
gaobinlong,
gbbafna,
jed326,
kotwanikunal,
mch2,
msfroh,
nknize,
owaiskazi19,
peternied,
reta,
Rishikesh1159,
sachinpkale,
saratvemulapalli,
shwetathareja,
sohami,
VachaShah,
jainankitk and
linuxpi
as code owners
|
❌ Gradle check result for dd198e3: 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? |
Signed-off-by: Divyansh Pandey <[email protected]>
pandeydivyansh1803
changed the title
|
❌ Gradle check result for 97d3bdb: 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? |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
[Describe what this change achieves]
Related Issues
Resolves #[Issue number to be closed when this PR is merged]
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.