search

lericson / simples3

Simple, quick Amazon AWS S3 interface in Python
BSD 2-Clause "Simplified" License
96 stars 36 forks source link

Added retries as a parameter #20

Closed pmav closed 9 years ago

pmav commented 9 years ago

I have added n_retries as a optional parameter. I'm using this lib to monitor performance of DreamHost Objects and sometimes I get the "ran out of retries" message. Most of the times 2 or 3 retries are enough.

lericson commented 9 years ago

Then I recommend that you set the attributes on the instance explicitly, there is no need for yet another keyword argument to this very huge constructor already. This is how you do it:

b = S3Bucket(…)
b.n_retries = 10
lericson commented 9 years ago

I would be happy to accept the James the documents this better. What do you think?

lericson commented 9 years ago

by "the James" I mean "a change"

pmav commented 9 years ago

Sounds, good, what you have in mind?

On 13/11/2014, at 20:56, Ludvig Ericson notifications@github.com wrote:

by "the James" I mean "a change"

— Reply to this email directly or view it on GitHub.

lericson commented 9 years ago

Is there no docs directory or something like that? :)

On 13 nov 2014, at 21:59, Pedro Verruma notifications@github.com wrote:

Sounds, good, what you have in mind?

On 13/11/2014, at 20:56, Ludvig Ericson notifications@github.com wrote:

by "the James" I mean "a change"

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub.