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

[Segment Replication] [Revert] Remove Restriction of strictly using only docrep for system indices and hidden indices (#8502) #8657

Merged
merged 1 commit into from
Jul 12, 2023

Conversation

Rishikesh1159
Copy link
Member

This reverts commit ef4881f.

Description

This reverts PR #8502 in 2.9 branch, as it might impact plugins with system indices

Related Issues

Resolves #[Issue number to be closed when this PR is merged]

Check List

  • New functionality includes testing.
    • All tests pass
  • New functionality has been documented.
    • New functionality has javadoc added
  • Commits are signed per the DCO using --signoff
  • Commit changes are listed out in CHANGELOG.md file (See: Changelog)

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.

…ndices and hidden indices. (opensearch-project#8502)"

This reverts commit ef4881f.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@Rishikesh1159 Rishikesh1159 added skip-changelog v2.9.0 'Issues and PRs related to version v2.9.0' labels Jul 12, 2023
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented Jul 12, 2023

Codecov Report

Merging #8657 (e3b64c5) into 2.9 (54eef1f) will decrease coverage by 0.51%.
The diff coverage is 90.00%.

@@             Coverage Diff              @@
##                2.9    #8657      +/-   ##
============================================
- Coverage     71.05%   70.54%   -0.51%     
+ Complexity    57461    57080     -381     
============================================
  Files          4745     4745              
  Lines        270757   270765       +8     
  Branches      39961    39963       +2     
============================================
- Hits         192390   191020    -1370     
- Misses        62154    63319    +1165     
- Partials      16213    16426     +213     
Impacted Files Coverage Δ
...h/cluster/metadata/MetadataCreateIndexService.java 79.49% <90.00%> (-0.07%) ⬇️

... and 455 files with indirect coverage changes

@kotwanikunal
Copy link
Member

kotwanikunal commented Jul 12, 2023

Do we need to backport this to 2.9?

NVM. Just saw the target branch.

@Rishikesh1159 Rishikesh1159 merged commit b717320 into opensearch-project:2.9 Jul 12, 2023
13 of 14 checks passed
r1walz pushed a commit to r1walz/OpenSearch that referenced this pull request Jul 13, 2023
…ndices and hidden indices. (opensearch-project#8502)" (opensearch-project#8657)

This reverts commit ef4881f.

Signed-off-by: Rishikesh1159 <rishireddy1159@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
skip-changelog v2.9.0 'Issues and PRs related to version v2.9.0'
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants