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 read/write in stream method for Diff Manifest when shard diff file is null #14947

Merged
merged 1 commit into from
Jul 24, 2024

Conversation

shailendra0811
Copy link
Contributor

@shailendra0811 shailendra0811 commented Jul 24, 2024

Backport 5744eae from #14938

@shailendra0811 shailendra0811 changed the title [Backport 2.x] Fix read/write method for Diff Manifest in case Shard diff file is null [Backport 2.x] Fix read/write in stream method for Diff Manifest when shard diff file is null Jul 24, 2024
Copy link
Contributor

✅ Gradle check result for c743c33: SUCCESS

Copy link

codecov bot commented Jul 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 71.50%. Comparing base (39da9bd) to head (c743c33).
Report is 2 commits behind head on 2.x.

Additional details and impacted files
@@             Coverage Diff              @@
##                2.x   #14947      +/-   ##
============================================
- Coverage     71.66%   71.50%   -0.16%     
+ Complexity    62739    62680      -59     
============================================
  Files          5120     5121       +1     
  Lines        294255   294266      +11     
  Branches      42877    42880       +3     
============================================
- Hits         210871   210415     -456     
- Misses        65668    66181     +513     
+ Partials      17716    17670      -46     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@Bukhtawar Bukhtawar merged commit 82c4c1e into opensearch-project:2.x Jul 24, 2024
59 of 60 checks passed
opensearch-trigger-bot bot pushed a commit that referenced this pull request Jul 24, 2024
…ll (#14947)

Signed-off-by: Shailendra Singh <[email protected]>
(cherry picked from commit 82c4c1e)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
dblock pushed a commit that referenced this pull request Jul 24, 2024
…ll (#14947) (#14951)

(cherry picked from commit 82c4c1e)

Signed-off-by: Shailendra Singh <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants