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

temporary fix for S3DDF/HDFSDDF not recognizing lower-case format names #358

Closed ubolonton closed 8 years ago

ubolonton commented 8 years ago

Description and related tickets, documents

Temporary fix for S3DDF and HDFSDDF not recognizing lower-case format names. E.g. It treated orc as unknown format, thus falling back to the default of csv.

Reviewers: @PangZhi @Huandao0812 @hai-adatao

Breaking changes & backward compatible issues

No

How to test

UT @hai-adatao We need to add more client-side tests for cases where specified format does not agree with file extensions (though we'd better remove the automatic format inference).

PR Progress

Make sure all checkboxes below are checked before merged