Add Backoff when restartable CatchupForever is executed

[R053NR07]

A customizable backoff is used when restarting the PartitionTables loading function.
Options for the view and processor can be used to customize the reset timeout and set the backoff provider.

[R053NR07]

@mattburman

[mattburman]

Looks good, it looks like I should be able to replicate our existing behaviour using https://github.com/jpillora/backoff

Presumably I'd just implement the goka.Backoff interface to use the existing library. We wouldn't need it to reset however, so I suppose we can just implement an empty function and not pass in the WithViewBackoffResetTimeout?

[mattburman]

Ignore the above, I misunderstood the purpose of WithViewBackoffResetTimeout. We would need to implement Reset so when the view reconnects, it will time out and then call reset so we can reset our backoff.

[R053NR07]

Ignore the above, I misunderstood the purpose of WithViewBackoffResetTimeout. We would need to implement Reset so when the view reconnects, it will time out and then call reset so we can reset our backoff.

Yes that was exactly what I intended :) But in the other case you described above I would wrap the actual backoff and add an empty reset function doing nothing. That way the reset would have been disabled.

Also I would like to add that github.com/jpillora/backoff already implements the Backoff interface. So you just have to hand it over as option and use it without further adoption.

[mattburman]

Also I would like to add that github.com/jpillora/backoff already implements the Backoff interface. So you just have to hand it over as option and use it without further adoption.

Ah I didn't realise that at first, nice one!