askterm comments

Repositories
Issues
Comments

Results 2 comments of


                                            askterm

For one retry, length of list of fixed intervals passed to NewSimpleBackoff should be 2

Sure, I will add a test. Thanks!

For one retry, length of list of fixed intervals passed to NewSimpleBackoff should be 2

In PerformRequestWithOptions, n is declared but [not initialised](https://github.com/olivere/elastic/blob/ca187de54e8160a36bd3e2a85cd30447d83b80ed/client.go#L1240). If there are no connections available, [n is incremented](https://github.com/olivere/elastic/blob/ca187de54e8160a36bd3e2a85cd30447d83b80ed/client.go#L1256). This means that while [retrying](https://github.com/olivere/elastic/blob/ca187de54e8160a36bd3e2a85cd30447d83b80ed/client.go#L1261), for first retry 1 is passed not 0....