Skip to content

[Backport 2.x] Improve exception handling in S3BlobContainer synchronous operations#17056

Merged
ashking94 merged 1 commit into2.xfrom
backport/backport-17049-to-2.x
Jan 20, 2025
Merged

[Backport 2.x] Improve exception handling in S3BlobContainer synchronous operations#17056
ashking94 merged 1 commit into2.xfrom
backport/backport-17049-to-2.x

Conversation

@opensearch-trigger-bot
Copy link
Copy Markdown
Contributor

@opensearch-trigger-bot opensearch-trigger-bot bot commented Jan 20, 2025

Backport 1b4a817 from #17049.

@github-actions
Improve exception handling in S3BlobContainer synchronous operations (#…
d57ef2d 
…17049)

Signed-off-by: Ashish Singh <ssashish@amazon.com>
(cherry picked from commit 1b4a817)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Jan 20, 2025

✅ Gradle check result for d57ef2d: SUCCESS

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 20, 2025 •
edited
Loading

Codecov Report

Attention: Patch coverage is 45.45455% with 6 lines in your changes missing coverage. Please review.

Project coverage is 71.81%. Comparing base (eee491f) to head (d57ef2d).
Report is 2 commits behind head on 2.x.

Files with missing lines Patch % Lines
...rg/opensearch/repositories/s3/S3BlobContainer.java 45.45% 5 Missing and 1 partial ⚠️
Additional details and impacted files 
@@             Coverage Diff              @@
##                2.x   #17056      +/-   ##
============================================
- Coverage     71.93%   71.81%   -0.13%     
+ Complexity    65673    65576      -97     
============================================
  Files          5318     5318              
  Lines        305838   305846       +8     
  Branches      44601    44602       +1     
============================================
- Hits         219991   219629     -362     
- Misses        67505    67878     +373     
+ Partials      18342    18339       -3

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

@ashking94 ashking94 merged commit 051d298 into 2.x Jan 20, 2025
@github-actions github-actions bot deleted the backport/backport-17049-to-2.x branch January 20, 2025 06:54
@opensearch-ci-bot opensearch-ci-bot mentioned this pull request Jan 20, 2025
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ashking94 ashking94 ashking94 approved these changes

@anasalkouz anasalkouz Awaiting requested review from anasalkouz anasalkouz is a code owner

@andrross andrross Awaiting requested review from andrross andrross is a code owner

@Bukhtawar Bukhtawar Awaiting requested review from Bukhtawar Bukhtawar is a code owner

@CEHENKLE CEHENKLE Awaiting requested review from CEHENKLE CEHENKLE is a code owner

@cwperks cwperks Awaiting requested review from cwperks cwperks is a code owner

@dblock dblock Awaiting requested review from dblock

@dbwiddis dbwiddis Awaiting requested review from dbwiddis dbwiddis is a code owner

@gbbafna gbbafna Awaiting requested review from gbbafna gbbafna is a code owner

@jainankitk jainankitk Awaiting requested review from jainankitk jainankitk is a code owner

@kotwanikunal kotwanikunal Awaiting requested review from kotwanikunal kotwanikunal is a code owner

@linuxpi linuxpi Awaiting requested review from linuxpi linuxpi is a code owner

@mch2 mch2 Awaiting requested review from mch2 mch2 is a code owner

@msfroh msfroh Awaiting requested review from msfroh msfroh is a code owner

@nknize nknize Awaiting requested review from nknize

@owaiskazi19 owaiskazi19 Awaiting requested review from owaiskazi19 owaiskazi19 is a code owner

@reta reta Awaiting requested review from reta reta is a code owner

@Rishikesh1159 Rishikesh1159 Awaiting requested review from Rishikesh1159 Rishikesh1159 is a code owner

@sachinpkale sachinpkale Awaiting requested review from sachinpkale sachinpkale is a code owner

@saratvemulapalli saratvemulapalli Awaiting requested review from saratvemulapalli saratvemulapalli is a code owner

@shwetathareja shwetathareja Awaiting requested review from shwetathareja shwetathareja is a code owner

@sohami sohami Awaiting requested review from sohami sohami is a code owner

@VachaShah VachaShah Awaiting requested review from VachaShah

Assignees

No one assigned

Labels

skip-changelog

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ashking94