ddf-project / DDF

Distributed DataFrame: Productivity = Power x Simplicity For Scientists & Engineers, on any Data Engine
http://ddf.io
Apache License 2.0
167 stars 42 forks source link

[No Jira] Don't try checking S3 account owner, which is not always permitted #368

Closed ubolonton closed 8 years ago

ubolonton commented 8 years ago

Description and related tickets, documents

Reviewers:

S3 credentials are no longer checked when creating a data source, only when used.

PR Progress

Make sure all checkboxes below are checked before merged

PangZhi commented 8 years ago

@ubolonton So now we removed host validation right?

ubolonton commented 8 years ago

@PangZhi What is host validation?

PangZhi commented 8 years ago

@ubolonton When you create datasource, validate it's valid or not.

ubolonton commented 8 years ago

It's credential validation, not host validation. And yes, this removes that.

PangZhi commented 8 years ago

Then i don't have other problem for this one.