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] [Segment Replication] Remove codec name string match check for checkpoints #7744

Merged
merged 2 commits into from
May 25, 2023

Conversation

opensearch-trigger-bot[bot]
Copy link
Contributor

Backport 89edd55 from #7741.

…oints (#7741)

* Remove codec name string match check for checkpoints

Signed-off-by: Poojita Raj <[email protected]>

* changelog added

Signed-off-by: Poojita Raj <[email protected]>

---------

Signed-off-by: Poojita Raj <[email protected]>
(cherry picked from commit 89edd55)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@Poojita-Raj
Copy link
Contributor

Poojita-Raj commented May 24, 2023

@dreamer-89
Copy link
Member

Resolved merge conflict on Changelog.md file from UI. Waiting for gradle check for merge now.

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

@codecov
Copy link

codecov bot commented May 25, 2023

Codecov Report

Merging #7744 (0d1aadf) into 2.x (bbbe203) will increase coverage by 0.16%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                2.x    #7744      +/-   ##
============================================
+ Coverage     70.52%   70.69%   +0.16%     
- Complexity    56395    56476      +81     
============================================
  Files          4665     4665              
  Lines        267683   267675       -8     
  Branches      39638    39636       -2     
============================================
+ Hits         188793   189237     +444     
+ Misses        62605    62151     -454     
- Partials      16285    16287       +2     
Impacted Files Coverage Δ
...in/java/org/opensearch/index/shard/IndexShard.java 69.97% <ø> (-0.36%) ⬇️
...ndices/replication/OngoingSegmentReplications.java 92.77% <ø> (-0.34%) ⬇️

... and 462 files with indirect coverage changes

@github-actions
Copy link
Contributor

Gradle Check (Jenkins) Run Completed with:

  • RESULT: UNSTABLE ❕
  • TEST FAILURES:
      1 org.opensearch.search.SearchWeightedRoutingIT.testStrictWeightedRoutingWithCustomString

@mch2 mch2 merged commit 6ea325c into 2.x May 25, 2023
@github-actions github-actions bot deleted the backport/backport-7741-to-2.x branch May 25, 2023 23:43
gaiksaya pushed a commit to gaiksaya/OpenSearch that referenced this pull request Jun 26, 2023
…oints (opensearch-project#7741) (opensearch-project#7744)

* Remove codec name string match check for checkpoints



* changelog added



---------


(cherry picked from commit 89edd55)

Signed-off-by: Poojita Raj <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Signed-off-by: Suraj Singh <[email protected]>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Suraj Singh <[email protected]>
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