You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2021/01/20 20:48:39 UTC

[GitHub] [airflow] RosterIn commented on a change in pull request #13796: Fix S3ToFTPOperator

RosterIn commented on a change in pull request #13796:
URL: https://github.com/apache/airflow/pull/13796#discussion_r561293394



##########
File path: airflow/providers/amazon/aws/transfers/s3_to_ftp.py
##########
@@ -27,12 +27,11 @@
 class S3ToFTPOperator(BaseOperator):
     """
     This operator enables the transferring of files from S3 to a FTP server.
-
     :param ftp_conn_id: The ftp connection id. The name or identifier for
         establishing a connection to the FTP server.
     :type ftp_conn_id: str
-    :param ftp_path: The ftp remote path. This is the specified file path for
-        uploading file to the FTP server.
+    :param ftp_path: The ftp remote path in where the file will be stored.
+        The desired filename must be specified here.

Review comment:
       ```suggestion
           The desired file name must be specified here.
   ```




----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org