Skip to content

Comments

Add circuit breaker pattern#1529

Closed
niveathika wants to merge 21 commits intoballerina-platform:masterfrom
niveathika:cb
Closed

Add circuit breaker pattern#1529
niveathika wants to merge 21 commits intoballerina-platform:masterfrom
niveathika:cb

Conversation

@niveathika
Copy link
Contributor

Purpose

Examples

Checklist

  • Linked to an issue
  • Updated the changelog
  • Added tests
  • Updated the spec
  • Checked native-image compatibility

@codecov
Copy link

codecov bot commented Jan 28, 2026

Codecov Report

❌ Patch coverage is 60.58201% with 149 lines in your changes missing coverage. Please review.
✅ Project coverage is 79.76%. Comparing base (2ab4ca2) to head (639e18a).

Files with missing lines Patch % Lines
.../ftp/client/circuitbreaker/FailureCategorizer.java 0.00% 62 Missing ⚠️
...dlib/ftp/client/circuitbreaker/CircuitBreaker.java 54.25% 35 Missing and 8 partials ⚠️
...tdlib/ftp/client/circuitbreaker/CircuitHealth.java 71.42% 14 Missing and 4 partials ⚠️
...java/io/ballerina/stdlib/ftp/client/FtpClient.java 79.22% 8 Missing and 8 partials ⚠️
...tp/client/circuitbreaker/CircuitBreakerConfig.java 89.58% 2 Missing and 3 partials ⚠️
...lib/ftp/client/circuitbreaker/FailureCategory.java 60.00% 3 Missing and 1 partial ⚠️
...erina/stdlib/ftp/client/circuitbreaker/Bucket.java 94.44% 1 Missing ⚠️

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

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1529      +/-   ##
============================================
- Coverage     81.70%   79.76%   -1.95%     
- Complexity      588      861     +273     
============================================
  Files            40       62      +22     
  Lines          2892     3850     +958     
  Branches        468      674     +206     
============================================
+ Hits           2363     3071     +708     
- Misses          337      503     +166     
- Partials        192      276      +84     

☔ View full report in Codecov by Sentry.
📢 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.

@niveathika niveathika force-pushed the cb branch 3 times, most recently from c098072 to 445615b Compare January 28, 2026 16:54
@niveathika niveathika added the Skip GraalVM Check This will skip the GraalVM compatibility check label Feb 1, 2026
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 1, 2026

@niveathika niveathika closed this Feb 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Skip GraalVM Check This will skip the GraalVM compatibility check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant