springml / spark-sftp

Spark connector for SFTP
Apache License 2.0
100 stars 98 forks source link

Support of FTP without secure layer #30

Closed mycaule closed 6 years ago

mycaule commented 6 years ago

Do you provide support for regular FTP?

Thank you.

samuel-pt commented 6 years ago

Currently not and we are planning it for next release. It requires changes in underlying library used by this connector. It would be great if you add this feature and create a PR

mycaule commented 6 years ago

Hello,

It looks like it is immediate with Spark lib.

Check out this code sample from the Databricks book. https://github.com/databricks/learning-spark/blob/master/src/main/scala/com/oreilly/learningsparkexamples/scala/BasicLoadTextFromFTP.scala

I think we could just write a fallback in your library to support FTP. You said you were planning it for next release, do you have ideas how to do it?

Or not support it at all, as jsch, which you rely on https://stackoverflow.com/a/7395178/1360476. Just need to say a few words about FTP in the README in this case.

samuel-pt commented 6 years ago

@mycaule Thanks for the info. Will update this library in the next release

mycaule commented 6 years ago

Better not use FTP.