[PR #551] [MERGED] Creating an abstraction of circuit breaker implementation #578

Closed
opened 2026-02-26 04:35:40 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/mageddo/dns-proxy-server/pull/551
Author: @mageddo
Created: 8/30/2024
Status: Merged
Merged: 9/2/2024
Merged by: @mageddo

Base: masterHead: feat/533-circuit-breaker-service-refactoring


📝 Commits (10+)

📊 Changes

21 files changed (+405 additions, -164 deletions)

View changed files

📝 RELEASE-NOTES.md (+32 -29)
📝 gradle.properties (+1 -1)
src/main/java/com/mageddo/commons/circuitbreaker/CircuitIsOpenException.java (+7 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/config/Config.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/mapper/ConfigMapper.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/config/validator/ConfigValidator.java (+1 -1)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerNonResilientService.java (+1 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerService.java (+2 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/FailsafeCircuitBreakerFactory.java (+62 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/OnCacheMustBeFlushedEvent.java (+21 -0)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactory.java (+16 -61)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFailSafeService.java (+2 -2)
📝 src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerPingCheckerService.java (+12 -14)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerDelegate.java (+13 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerDelegateFailsafe.java (+38 -0)
src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerExecutorService.java (+33 -0)
src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/FailsafeCircuitBreakerFactoryTest.java (+73 -0)
📝 src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactoryTest.java (+54 -49)
📝 src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFailSafeServiceCompTest.java (+1 -1)
📝 src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerPingCheckerServiceTest.java (+5 -4)

...and 1 more files

📄 Description

Relates to #533


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/mageddo/dns-proxy-server/pull/551 **Author:** [@mageddo](https://github.com/mageddo) **Created:** 8/30/2024 **Status:** ✅ Merged **Merged:** 9/2/2024 **Merged by:** [@mageddo](https://github.com/mageddo) **Base:** `master` ← **Head:** `feat/533-circuit-breaker-service-refactoring` --- ### 📝 Commits (10+) - [`e62317b`](https://github.com/mageddo/dns-proxy-server/commit/e62317bab05f0e889a4cafbff6a2c7979d6a4ea3) setup resilience4j - [`742a0e8`](https://github.com/mageddo/dns-proxy-server/commit/742a0e8bf9c5b386a61827807eea7a13a7fc4601) specifying circuit breaker strategy name - [`6ab3d76`](https://github.com/mageddo/dns-proxy-server/commit/6ab3d76c2d874a74a599cf4c845623ea20ec567c) creating agnostic interface to support multiple circuit breaker strategy types - [`460233a`](https://github.com/mageddo/dns-proxy-server/commit/460233aafc6d93b24c27093f7f9b3b8957dafdf1) fixme - [`d9c2852`](https://github.com/mageddo/dns-proxy-server/commit/d9c28521cc4d07f726b1628888a7d630cf44e482) sppliting circuit breaker factory - [`d05ff95`](https://github.com/mageddo/dns-proxy-server/commit/d05ff957555d8aacc5f8fd93ea39a7584eb4daba) refactoring is done - [`a7dd4c0`](https://github.com/mageddo/dns-proxy-server/commit/a7dd4c0fd5d02b8ab88fea69968a84e02be296b5) test is passing - [`5869154`](https://github.com/mageddo/dns-proxy-server/commit/58691540a68fbf22d183682d1d34a3e327c29435) test is passing - [`34544f2`](https://github.com/mageddo/dns-proxy-server/commit/34544f21e25737ac9b43e2d4d6c7b557455e50e1) test is passing - [`fda3d97`](https://github.com/mageddo/dns-proxy-server/commit/fda3d972d1a851844d7c7a617253cd324a9b9335) refactoring package ### 📊 Changes **21 files changed** (+405 additions, -164 deletions) <details> <summary>View changed files</summary> 📝 `RELEASE-NOTES.md` (+32 -29) 📝 `gradle.properties` (+1 -1) ➕ `src/main/java/com/mageddo/commons/circuitbreaker/CircuitIsOpenException.java` (+7 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/Config.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/mapper/ConfigMapper.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/config/validator/ConfigValidator.java` (+1 -1) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerNonResilientService.java` (+1 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/CircuitBreakerService.java` (+2 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/FailsafeCircuitBreakerFactory.java` (+62 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/OnCacheMustBeFlushedEvent.java` (+21 -0) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactory.java` (+16 -61) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFailSafeService.java` (+2 -2) 📝 `src/main/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerPingCheckerService.java` (+12 -14) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerDelegate.java` (+13 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerDelegateFailsafe.java` (+38 -0) ➕ `src/main/java/com/mageddo/dnsproxyserver/solver/remote/circuitbreaker/application/CircuitBreakerExecutorService.java` (+33 -0) ➕ `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/FailsafeCircuitBreakerFactoryTest.java` (+73 -0) 📝 `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFactoryTest.java` (+54 -49) 📝 `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerFailSafeServiceCompTest.java` (+1 -1) 📝 `src/test/java/com/mageddo/dnsproxyserver/solver/remote/application/failsafe/CircuitBreakerPingCheckerServiceTest.java` (+5 -4) _...and 1 more files_ </details> ### 📄 Description Relates to #533 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 04:35:40 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/dns-proxy-server-mageddo#578
No description provided.