Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update goamz import #30

Open
toffer opened this issue Feb 2, 2015 · 7 comments
Open

Update goamz import #30

toffer opened this issue Feb 2, 2015 · 7 comments

Comments

@toffer
Copy link
Contributor

toffer commented Feb 2, 2015

I noticed that the aws package we use is no longer maintained at its current repo, but instead has moved to a new one:

https://github.com/crowdmob/goamz -> https://github.com/AdRoll/goamz

We should update our imports accordingly.

@matthewkmayer
Copy link

To muddy the waters, what about the other versions available?

Or the official alpha/experimental Golang SDK? https://github.com/awslabs/aws-sdk-go

@amjith
Copy link
Contributor

amjith commented Mar 24, 2015

The description on their github repo An incredibly experimental, automatically generated set of AWS clients in Go. That really doesn't instill confidence. :)

But if it ever becomes the official client, we can switch to using that.

@joeshaw
Copy link

joeshaw commented Apr 28, 2015

To complicate things even further, there is a library that does parallel transfers to/from S3, which might speed things up: https://github.com/rlmcpherson/s3gof3r

@relistan
Copy link
Contributor

@joeshaw That's cool. But, we're already doing that ourselves inside Dogestry :)

@didip
Copy link
Contributor

didip commented Jun 4, 2015

@joeshaw PR #58 will introduce s3gof3r. I think it is a good idea if we move on to https://github.com/aws/aws-sdk-go for the rest of the features.

@matthewkmayer
Copy link

Is there still interest in using the newer goamz library or should we use with the official lib since it's out of experimental status?

@didip
Copy link
Contributor

didip commented Jul 13, 2015

I think we should use the official lib. Seems like the community is migrating towards the official lib.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

6 participants