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

Added support to eu-central-1 region in S3 #513

Merged
merged 1 commit into from
Aug 2, 2016
Merged

Added support to eu-central-1 region in S3 #513

merged 1 commit into from
Aug 2, 2016

Conversation

anmolnar
Copy link

Adds support of "eu-central-1" region to S3 file system. Lets Priam work in Frankfurt AWS region.

@tulumvinh tulumvinh merged commit 77c9834 into Netflix:3.x Aug 2, 2016
@tulumvinh
Copy link
Contributor

Dolphy17, would you create a PR to the 2.x branch?

I plan to eliminate the need to have both 3.x and 2.x. I want to only have 2.x (for Cassandra 2.x versions) but I need to implement fetching the Cassandra version first (once you know the version, you can execute the applicable code).

@anmolnar
Copy link
Author

anmolnar commented Aug 3, 2016

Sure, I can do it. Would you also like me to add support for the other european region as well "eu-west-1"?
Also, I think I forgot to add unit tests, are there any present for the US regions?

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

Successfully merging this pull request may close these issues.

2 participants