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] Bugfix/496 rollover alias with filters #4651

Merged
merged 1 commit into from
Oct 6, 2022

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 54f8fdd from #4499

* 496: Added filters, indexRouting and searchRouting when doing rollover on alias

Signed-off-by: Stevan Buzejic <buzejic.stevan@gmail.com>

* 496: Removed unused map property in metada alias rollover tests

Signed-off-by: Stevan Buzejic <buzejic.stevan@gmail.com>

Signed-off-by: Stevan Buzejic <buzejic.stevan@gmail.com>
Co-authored-by: Stevan Buzejic <stevan.buzejic@htecgroup.com>
(cherry picked from commit 54f8fdd)
@opensearch-trigger-bot opensearch-trigger-bot bot requested review from a team and reta as code owners September 30, 2022 16:06
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2022

Gradle Check (Jenkins) Run Completed with:

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2022

Gradle Check (Jenkins) Run Completed with:

@codecov-commenter
Copy link

Codecov Report

Merging #4651 (9e784ad) into 2.x (bd532e1) will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##                2.x    #4651      +/-   ##
============================================
- Coverage     70.70%   70.63%   -0.07%     
+ Complexity    57157    57150       -7     
============================================
  Files          4585     4585              
  Lines        274523   274536      +13     
  Branches      40232    40233       +1     
============================================
- Hits         194099   193930     -169     
- Misses        64180    64362     +182     
  Partials      16244    16244              
Impacted Files Coverage Δ
...dmin/indices/rollover/MetadataRolloverService.java 93.00% <100.00%> (+2.02%) ⬆️
...a/org/opensearch/cluster/metadata/AliasAction.java 83.33% <100.00%> (+0.98%) ⬆️
...g/opensearch/index/analysis/CharFilterFactory.java 0.00% <0.00%> (-100.00%) ⬇️
...java/org/opensearch/client/indices/DataStream.java 0.00% <0.00%> (-76.09%) ⬇️
...n/indices/forcemerge/ForceMergeRequestBuilder.java 0.00% <0.00%> (-75.00%) ⬇️
.../opensearch/client/indices/CloseIndexResponse.java 17.50% <0.00%> (-73.75%) ⬇️
.../indices/forcemerge/TransportForceMergeAction.java 25.00% <0.00%> (-58.34%) ⬇️
...ava/org/opensearch/action/NoSuchNodeException.java 0.00% <0.00%> (-50.00%) ⬇️
...ch/transport/ReceiveTimeoutTransportException.java 50.00% <0.00%> (-50.00%) ⬇️
...cluster/coordination/PendingClusterStateStats.java 20.00% <0.00%> (-48.00%) ⬇️
... and 496 more

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

@xuezhou25 xuezhou25 merged commit 36925c5 into 2.x Oct 6, 2022
@xuezhou25 xuezhou25 deleted the backport/backport-4499-to-2.x branch October 6, 2022 23:25
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.

3 participants