Code review comment for lp:~dave-cheney/goamz/004-send-409-conflict

Revision history for this message
John A Meinel (jameinel) wrote :

On 2012/12/10 23:40:16, dfc wrote:
> On 2012/12/10 23:39:03, dfc wrote:
> > Please take a look.

> Thanks for your comments. I have added a *Config style structure which
can be
> extended when we need more control over the s3test.Server.

LGTM. If you want, we could have a "useast1_config" and a "uswest1" or
whatever. So we don't have to figure out what the various configs
actually mean. But overall it seems fine to me.

https://codereview.appspot.com/6901061/

« Back to merge proposal