Skip to content

Getting rid of log-and-throw constructions - #712

Open
Myllyenko wants to merge 1 commit into
ydb-platform:masterfrom
Myllyenko:codex/fix-log-and-throw
Open

Getting rid of log-and-throw constructions#712
Myllyenko wants to merge 1 commit into
ydb-platform:masterfrom
Myllyenko:codex/fix-log-and-throw

Conversation

@Myllyenko

Copy link
Copy Markdown

Removes log-and-throw constructions that could cause the same failure to be logged multiple times—typically twice: once in the library and once in the top-level application.

@Myllyenko Myllyenko changed the title Getting rid of log-and-throw Redundant logs were removed Getting rid of log-and-throw constructions Aug 25, 2026
@codecov

codecov Bot commented Aug 25, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 0% with 1 line in your changes missing coverage. Please review.
✅ Project coverage is 72.34%. Comparing base (bc6886d) to head (e31863e).

Files with missing lines Patch % Lines
...in/java/tech/ydb/core/operation/OperationTray.java 0.00% 1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (0.00%) is below the target coverage (80.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #712      +/-   ##
============================================
- Coverage     72.42%   72.34%   -0.08%     
+ Complexity     3522     3516       -6     
============================================
  Files           391      391              
  Lines         16341    16327      -14     
  Branches       1702     1702              
============================================
- Hits          11835    11812      -23     
- Misses         3867     3872       +5     
- Partials        639      643       +4     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant