OpenFeign / feign

Feign makes writing java http clients easier
Apache License 2.0
9.44k stars 1.92k forks source link

Add description for RetryableException #2459

Closed heli-os closed 3 months ago

heli-os commented 3 months ago

Hi @velo ,

I've been using RetryableException, it's confusing because of last changes.

following issue: https://github.com/OpenFeign/feign/issues/2458

It's not intuitive to me that a Wrapper Long type doesn't retry on null when looking at the constructor.

existing constructors use null inside the constructor for special purposes.

The overload resolution ambiguity in the mentioned issue is beyond my control right now, but it would be nice to at least minimize confusion when using the new constructor.

thanks :)