You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by "James Meickle (JIRA)" <ji...@apache.org> on 2019/03/07 14:14:00 UTC

[jira] [Created] (AIRFLOW-4040) SSHHook and SFTPHook have different SFTP implementations

James Meickle created AIRFLOW-4040:
--------------------------------------

             Summary: SSHHook and SFTPHook have different SFTP implementations
                 Key: AIRFLOW-4040
                 URL: https://issues.apache.org/jira/browse/AIRFLOW-4040
             Project: Apache Airflow
          Issue Type: Improvement
    Affects Versions: 1.10.1
            Reporter: James Meickle


Both of these hooks are capable of opening SFTP connections (one is used in the SFTPSensor, another in the SFTPOperator). However, they have different implementations. We got bitten by this, where one of the hooks worked on a particular SFTP server, and the other did not (because they initialize the connection in slightly different ways).

I think that if possible, it makes more sense to drop pysftp as a dependency, and have both operators use Paramiko's SFTP support via the SSH hook.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)