On Mon, Oct 28, 2019 at 9:02 AM Oleg Kalnichevski <[email protected]> wrote:
> On Mon, 2019-10-28 at 12:10 +0100, Michael Osipov wrote: > > ... > > > We are still in beta, why not? Or make a generic interface > > RetryStrategy > > and make ServiceUnavailableRetryStrategy extend it with marked as > > deprecated. I was really confused by the name. According to it, the > > only > > viable status code is 503. > > > > Michael > > > > Yes, we are technically still in BETA but the next release might be a > GA. Anyway deprecation of ServiceUnavailableRetryStrategy in favor of a > generic RetryStrategy sounds good like a good plan to me. > +1 to renaming. Deprecating one type for another while in Beta is, to me, absurd. There is no cost to changing as the IDE does all the work. Downstream call sites are using a Beta so know what they are in for: beta code. I do not think one can -1 doing nothing but it looks bad to knowingly leave cruft behind when not required to do so. Gary > > Oleg > > > --------------------------------------------------------------------- > To unsubscribe, e-mail: [email protected] > For additional commands, e-mail: [email protected] > >
