Add option to customize expected exceptions in DefaultEndpoint
#2830
Labels
status: waiting-for-feedback
We need additional information before we can continue
Feature Request
Is your feature request related to a problem? Please describe
We’ve implemented a circuit breaker using a Netty Customizer as suggested in #1530 discussion. We now observe a large volume of logs from
DefaultEndpoint
when the circuit breaker is open, because the command is completed exceptionally with an exception class that is not expected by this code block.Describe the solution you'd like
We’d like to specify in
ClientOptions
a set of exception classes that should not be logged, similar to howClosedChannelException
is skipped. A drawback of this is additional complexity for a relatively advanced use case.Describe alternatives you've considered
We considered wrapping our internal exception in
ClosedChannelException
. I believe this would work fine with the current Lettuce implementation, but it seems incorrect to do this, because aChannel
that closes cannot re-open, and the circuit breaker will very likely permit future calls on the same channel.Teachability, Documentation, Adoption, Migration Strategy
This would be opt-in—the default client option would be an empty set.
The text was updated successfully, but these errors were encountered: